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

Remove unused dependencies: greenlet, cffi #403

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

Conversation

gdrosos
Copy link

@gdrosos gdrosos commented Aug 29, 2023

Summary

This pull request addresses the removal of the unused dependencies greenlet and cffi from the project. It's part of an ongoing research endeavor focusing on the identification and elimination of code bloat within software projects.

Rationale

The greenlet dependency was added in 180a65e but remains unused, serving only as a transitive dependency. Similarly, the cffi dependency, added in 2208815, is no longer required due to updates in the cryptography dependencies.

Removing these unused dependencies contributes to a leaner codebase, reduces security risks, and simplifies dependency management.

Changes

  • Removed greenlet and cffi dependencies from requirements.in.
  • Updated requirements3.txt to reflect the removed dependencies.

Impact

  • Decreased package size
  • Enhanced project maintainability

I also signed the Contributor License Agreement (CLA) according to the contributing guidelines

@gdrosos
Copy link
Author

gdrosos commented Sep 7, 2023

Hello @ramonpetgrave64, @skiptomyliu, @alejandroroiz

I hope this message finds you well. We are currently engaged in a research project focused on optimizing projects by identifying and eliminating unnecessary dependencies. Your insights and expertise as a maintainer would greatly contribute to our research efforts.

Could you please take a moment to review these changes? Your input would be highly valuable to us.

Thank you for your consideration!

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.

1 participant