python-seo-analyzer icon indicating copy to clipboard operation
python-seo-analyzer copied to clipboard

Remove Unused Dependency: Requests

Open gdrosos opened this issue 2 years ago • 1 comments

Summary

This pull request removes the unused dependency requests from the requirements.txt file and updates the corresponding note in the README. This change is a result of ongoing research aimed at identifying and eliminating code bloat within software projects.

Rationale

The requests library was previously used within the project but was deemed unnecessary and removed in favor of urllib3 in this pull request. Despite the transition to urllib3, requests remained listed as a requirement in the requirements.txt file, and a related note in the README was left outdated. Removing this unused dependency and updating the documentation is aligned with best practices for dependency management and user guidance.

Changes

  • Removed the requests dependency from the requirements.txt file.
  • Updated the README to remove the note pertaining to requests.exceptions.SSLError, as it is no longer relevant.

Impact

  • Reduced Package Size: The removal of this unused dependency will lead to a decrease in the overall size of the installed packages.
  • Updated Documentation: The README will now accurately reflect the current state of the codebase, providing correct guidance to users and developers.
  • Simplified Dependency Tree: Fewer dependencies make the project easier to maintain and can speed up installation.

gdrosos avatar Aug 10 '23 15:08 gdrosos

Hello @sethblack, @gnovaro ,

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!

gdrosos avatar Aug 27 '23 14:08 gdrosos

Thank you! Removed.

sethblack avatar Apr 21 '24 18:04 sethblack