Remove transitive dependency 'ply' from requirements.txt by vtalos · Pull Request #517 · dropbox/dropbox-sdk-python · GitHub
Skip to content

Remove transitive dependency 'ply' from requirements.txt#517

Open
vtalos wants to merge 1 commit into
dropbox:mainfrom
vtalos:remove-transitive-ply
Open

Remove transitive dependency 'ply' from requirements.txt#517
vtalos wants to merge 1 commit into
dropbox:mainfrom
vtalos:remove-transitive-ply

Conversation

@vtalos

@vtalos vtalos commented Jul 17, 2025

Copy link
Copy Markdown

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 by stone, is specified as a requirement in the requirements.txt file, 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.txt to let pip manage it automatically, which helps keeping the dependency list clean.

Hope this is helpful!

Checklist

General Contributing

  • Have you read the Code of Conduct and signed the CLA?

Is This a Code Change?

  • Non-code related change (markdown/git settings etc)
  • SDK Code Change
  • Example/Test Code Change

Validation

  • Does tox pass?
  • Do the tests pass?

@CLAassistant

Copy link
Copy Markdown

@alyx-db

alyx-db commented Jul 21, 2025

Copy link
Copy Markdown

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.

3 participants