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

Upgrade to Poetry build system for dependency management #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dartt0n
Copy link

@dartt0n dartt0n commented Nov 19, 2024

The current project relies on requirements.txt to manage dependencies, with versions strictly fixed up to the minor version, which is considered a bad practice. This is because other users will not be able to install this package due to version conflicts.

I suggest switching to using Poetry as a build system, that offers several advantages that can significantly improve development workflow and project maintainability by solving transitive dependencies problem.

I have also noted that the code in some areas is not well-formatted. I fixed that using black.

Please review the changes and provide your feedback on them.

@dartt0n
Copy link
Author

dartt0n commented Nov 19, 2024

Also, please notice, I have added myself to the package contributors:

Let me know, if you are okay with that

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.

1 participant