Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Poetry Support #184

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Add Poetry Support #184

wants to merge 18 commits into from

Conversation

DanielAvdar
Copy link

@DanielAvdar DanielAvdar commented Feb 15, 2024 •
edited

Migrate to Poetry for packaging and dependency management #156

@DanielAvdar DanielAvdar marked this pull request as ready for review February 15, 2024 17:54
@DanielAvdar
supress mypy attr not define, caused by socket lazy import.
389bae9 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar had a problem deploying to fork-pr-integration-tests February 16, 2024 08:48 — with GitHub Actions Failure
@DanielAvdar
update run_checks.yaml workflow with poetry unittests.
fee9efc 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar had a problem deploying to fork-pr-integration-tests February 16, 2024 08:56 — with GitHub Actions Failure
@DanielAvdar
refactored pyproject.toml.
bc50cef 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar
second attempt create extra requirement scrapy.
9f172ff 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar
include extra dependencies for poetry install.
6a73892 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar
include extra dependencies for poetry install.
d2b6f29 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar
include extra dependencies for poetry install.
1795f18 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar had a problem deploying to fork-pr-integration-tests February 16, 2024 18:32 — with GitHub Actions Failure
@DanielAvdar
updated unittests ci with poetry implementation.
8054c17 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar had a problem deploying to fork-pr-integration-tests February 16, 2024 19:14 — with GitHub Actions Failure
@DanielAvdar
schema fix.
2589fd2 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar had a problem deploying to fork-pr-integration-tests February 17, 2024 13:43 — with GitHub Actions Failure
@vdusek
Copy link
Contributor

vdusek commented Feb 20, 2024

Hi @DanielAvdar, migration to Poetry was planned in Q2-Q3 of this year, after we thoroughly tested it in our internal repositories. Also, I believe twine could be replaced by poetry as well. Definitely, thank you for the contribution, however, we'll come back to this PR in a few months, once migration to Poetry in SDK is on the table.

@DanielAvdar
Copy link
Author

Hi @DanielAvdar, migration to Poetry was planned in Q2-Q3 of this year, after we thoroughly tested it in our internal repositories. Also, I believe twine could be replaced by poetry as well. Definitely, thank you for the contribution, however, we'll come back to this PR in a few months, once migration to Poetry in SDK is on the table.

@vdusek Indeed, Twine can be replaced. This PR simply adds Poetry support, and everything else should work the same as before.

@DanielAvdar
fix build with poetry.
f0bbc86 
Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar had a problem deploying to fork-pr-integration-tests February 20, 2024 08:12 — with GitHub Actions Failure
@DanielAvdar DanielAvdar changed the title Poetry Migration Add Poetry Support Feb 20, 2024
@B4nan B4nan added the t-tooling Issues with this label are in the ownership of the tooling team. label Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@DanielAvdar @vdusek @B4nan