Skip to content

Move to new PEP 517 standard with poetry and more "meta" package updates#270

Merged
upbit merged 8 commits intoupbit:masterfrom
Nachtalb:na/feature-poetry
Sep 17, 2023
Merged

Move to new PEP 517 standard with poetry and more "meta" package updates#270
upbit merged 8 commits intoupbit:masterfrom
Nachtalb:na/feature-poetry

Conversation

@Nachtalb
Copy link
Copy Markdown
Contributor

This pull request migrates the project from using setuptools to Poetry, in line with PEP 517, to improve the project's overall development experience and maintainability. The key enhancements include:

  • Streamlined dependency management, ensuring consistent and deterministic builds across environments.
  • Simplified package publishing, reducing the complexity of deployment.
  • Seamless integration with virtual environments, improving isolation and consistency.

With this migration, the project benefits from a more robust build process, ultimately contributing to higher code quality and a smoother development workflow.

In addition to poetry, I have also updated the remaining "meta" packages (mypy, black, pre-commit etc.):

  • Upgraded pre-commit modules to their latest versions, ensuring adherence to current best practices and coding standards.
  • Performed an update of all code files to ensure they pass the newly configured pre-commit checks, enhancing overall code quality and consistency.
  • Replaced Flake with Ruff, a modern and extensible linting tool, for more accurate and configurable code analysis. This switch allows us to maintain a high level of code quality while also tailoring the linter to our project's specific needs.

Switching to Poetry (the new defacto standard), compliant with PEP 517, for improved dependency handling, deterministic build process, and easier maintainability.
@Nachtalb Nachtalb force-pushed the na/feature-poetry branch 3 times, most recently from f190fec to e5ebdd0 Compare April 25, 2023 20:25
@Nachtalb Nachtalb force-pushed the na/feature-poetry branch from e5ebdd0 to d71eeb1 Compare April 25, 2023 20:36
Copy link
Copy Markdown
Contributor

@nautics889 nautics889 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Nachtalb
Anyway, checks are passed, also i've tested building project environment with Python 3.7.16 and 3.10.11, everything went well.
+1
Thank you!

@Nachtalb Nachtalb force-pushed the na/feature-poetry branch from 11e7b7e to b8150a0 Compare April 26, 2023 22:55
@Nachtalb Nachtalb requested a review from nautics889 April 26, 2023 23:00
Copy link
Copy Markdown
Contributor

@nautics889 nautics889 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Nachtalb
Great, thank you!

@upbit upbit merged commit 734a598 into upbit:master Sep 17, 2023
@upbit
Copy link
Copy Markdown
Owner

upbit commented Sep 17, 2023

Thanks for the contribution!

Sorry for late response, merged.

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.

3 participants