The project has build, test, and install errors due to wrong choices and neglected maintenance. This PR addresses all the known problems so far.
Change description
Rewrites all HTTP requests recording in pytest-recording/vcrpy
Pin all dependencies to avoid dependencies drift
Rewrites all HTTP requests recording in pytest-recording/vcrpy
pytest-recording/vcrpy record HTTP request body and string instead of base64. For some reasons, when toggl-track runs on Python 3.10+ it is no longer able to decode the bas64 successfully. I guess this error is due to a mix of dependencies.
Pin all dependencies to avoid dependencies drift
Setting no constraints for dependencies is a risky business.
Additional Notes
Reviewer checklist
[x] PR address a single concern.
[x] PR title and description are appropriately filled.
[x] Changes will be merged in main.
[x] Changes are covered by tests.
[x] Logging is meaningful in case of troubleshooting.
[ ] Docs are updated (at least the README.md, if needed).
[x] History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well-formatted.
zmoog
commented
6 months ago
Motivation
The project has build, test, and install errors due to wrong choices and neglected maintenance. This PR addresses all the known problems so far.
Change description
Rewrites all HTTP requests recording in pytest-recording/vcrpy
pytest-recording/vcrpy record HTTP request body and string instead of base64. For some reasons, when toggl-track runs on Python 3.10+ it is no longer able to decode the bas64 successfully. I guess this error is due to a mix of dependencies.
Pin all dependencies to avoid dependencies drift
Setting no constraints for dependencies is a risky business.
Additional Notes
Reviewer checklist
main.Docs are updated (at least theREADME.md, if needed).CONTRIBUTING.md) and are well-formatted.