-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
This project has a cyclic dependency to its self in setup.py
#19
Comments
This project is very experimental and hasn't been developed in some time. Thanks for picking this up. This is not intentional. Happy to take a PR to fix it if you'd like to contribute to it, or I can make the change. Thanks! |
…=[...]) ... breaking eg `pipdeptree` dues to cycle detected in working-set dependencies.
Did a PR. |
That's how open source projects improve! Thanks for the efforts. Much appreciated. Will merge it this weekend. |
btw, if you are using this library and are interested in developing it further, please do feel free to submit other PR. open to new ideas and expansion / revision of source. |
fix(#19): was including myself in setup(install_requires=[...])
Can this please be released? It causes assertion errors when installed with Poetry. |
I am out of the office until 07.02.2020.
Note: This is an automated response to your message "Re:
[chrissimpkins/naked] This project has a cyclic dependency to its self in
`setup.py` (#19)" sent on 06.02.2020 02:47:43.
This is the only notification you will receive while this person is away.
|
Can we get the latest PR pushed as a release? |
@chrissimpkins A new release with this PR would help a lot of people out. |
This prevents popular packages (like Cypto) from installing with Poetry. We should probably do that release and not break every package that has this as a dependency. |
Done https://pypi.org/project/Naked/ Apologies for the time that it took to address this. |
Thanks for the changes to address this Kostis. |
For
v0.1.31
i get this fro `pipdeptree:And indeed, this project exists in its "install_requires` list.
Is that on purpose?
The text was updated successfully, but these errors were encountered: