I hope you're doing well! I've just opened this pull request that proposes the removal of the unused dependency xmltodict from the setup.py configuration file. It's part of an ongoing research endeavor focusing on the identification and elimination of code bloat within software projects. Your insights on this would be really valuable.
Rationale
The xmltodict package was added in e8dea47, and was used in googleads/adwords.py. However, this file was later removed, and upon analysis of the codebase, it was found that it is not currently being utilized within the project. Removing this unused dependency can reduce the overall footprint of the application, mitigate potential security risks, and simplify the dependency management process.
Changes
Removed the dependency to the xmltodict PyPI package from the `setup.py file.
Removed the corresponding dependnency requirement from the README.
Impact
Reduced package size: The removal of this unused dependency will lead to a decrease in the overall size of the installed packages.
Simplified dependency tree: Fewer dependencies make the project easier to maintain and can speed up installation.
I have also signed a individual CLA, according to CONTRIBUTING.md
Summary
Hello @christopherseeley, @msaniscalchi,
I hope you're doing well! I've just opened this pull request that proposes the removal of the unused dependency
xmltodictfrom thesetup.pyconfiguration file. It's part of an ongoing research endeavor focusing on the identification and elimination of code bloat within software projects. Your insights on this would be really valuable.Rationale
The
xmltodictpackage was added in e8dea47, and was used ingoogleads/adwords.py. However, this file was later removed, and upon analysis of the codebase, it was found that it is not currently being utilized within the project. Removing this unused dependency can reduce the overall footprint of the application, mitigate potential security risks, and simplify the dependency management process.Changes
xmltodictPyPI package from the `setup.py file.Impact
I have also signed a individual CLA, according to
CONTRIBUTING.md