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

Install notifypy CLI script with poetry as well #50

Merged
merged 1 commit into from Jan 3, 2023

Conversation

wisnij
Copy link
Contributor

@wisnij wisnij commented Jan 2, 2023

It looks like the switch from setuptools to poetry broke the CLI script install, as that was only specified in setup.py. This change adds the appropriate [tool.poetry.scripts] entry to pyproject.toml so that will happen again:

~/code/notify-py (wisnij/20230102-poetry-install-cli=) $ pip install . -v
Using pip 22.3.1 from /usr/lib/python3/dist-packages/pip (python 3.10)
[...]
Successfully built notify-py
Installing collected packages: notify-py
  changing mode of /home/wisnij/.local/bin/notifypy to 755
Successfully installed notify-py-0.3.38

@ms7m ms7m merged commit 6f61bc1 into ms7m:master Jan 3, 2023
@ms7m
Copy link
Owner

ms7m commented Jan 3, 2023

Merged, Thanks for the lookout! 🎉

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