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.

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

Poetry migration example #186

Closed
wants to merge 36 commits into from

Conversation

erickisos
Copy link
Contributor

@erickisos erickisos commented Jan 24, 2022

Status

WIP

Background context

Description of the changes proposed in the pull request

This is a major PR to show which would be the full change of the project if we decide to migrate everything to Poetry, we would need to add some extra changes to documentation to keep everything synced, however the major advantage is the centralized file for tool configuration and dependency management, also and just as a reference, using this would lead us eventually to having a more reliable system for multiple python versions.

Where should the reviewer start?

The main difference is on the side of config files, for example, now the pyproject.toml is the main source of configuration for the majority of tools (let's say mypy, pytest), we have included the files for Github Actions because of the problem with CircleCI, and you can check the new workflows on .github/workflows.

Remaining problems or questions

TBD(?)

Relates to:

@erickisos erickisos requested a review from a team as a code owner January 24, 2022 17:35
@erickisos erickisos marked this pull request as draft January 24, 2022 17:36
@erickisos erickisos force-pushed the poetry-migration-example branch from edac95c to 9a50df7 Compare February 4, 2022 16:03
@erickisos
Copy link
Contributor Author

Now we have the full pipeline running with Poetry and Github Actions here

@erickisos erickisos marked this pull request as ready for review February 4, 2022 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants