- Notifications
You must be signed in to change notification settings - Fork329
Remove transitive dependency 'ply' from requirements.txt#517
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
CLAassistant commentedJul 17, 2025
|
alyx-db commentedJul 21, 2025
Hi@vtalos, Thanks for your submission. I'll pass this along to the team. I can't promise if or when that might be implemented though. |
As part of our ongoing research on Python dependency management we noticed a potential improvement in your project’s dependency management.
Specifically, the transitive dependency
ply, which is required bystone, is specified as a requirement in therequirements.txtfile, even though it is not used directly and does not need to be listed explicitly, as it will be automatically handled by pip during installation.This PR removes it from
requirements.txtto let pip manage it automatically, which helps keeping the dependency list clean.Hope this is helpful!
Checklist
General Contributing
Is This a Code Change?
Validation
toxpass?