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

Migrate to Poetry #32

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

Conversation

RobertRosca
Copy link

I was interested in this package and when trying to install via pipx noticed that none of the dependencies were installed due to #17 (fixed by #30).

Also the requirements weren't pinned and since this is a self-contained application, not a library meant to be imported, pinning all dependencies seemed like a good idea.

To do that I converted the project from setup.py to Poetry with a pyproject.toml file, committed the poetry.lock file so that the pinned packages are used, and changed the way the command line argument is exposed so that termpdf is added to bin during installation of the package.

Updated the readme to cover the new installation instructions.

Not sure if this would be accepted as it is a large change to how the project is packaged, so feel free to reject if you prefer setup.py.

Copy link

@renzmann renzmann left a comment

Choose a reason for hiding this comment

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

I think this is definitely preferable to what I did in #30. @dsanson it looks like at least a few people have had the same dependency problem based on this PR and the one I linked above. I think the migration proposed here would be worth it.

README.md Outdated Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
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.

None yet

2 participants