Dependencies: Modernize to prepare unlocking more recent Python versions #543
Conversation
| "testing": [ | ||
| "tox==3.14.2", | ||
| "pytest-freezegun==0.4.2", | ||
| "pytest<9", | ||
| "pytest-cov<7", | ||
| "pytest-freezer<0.5", | ||
| "pytest-random-order<2", | ||
| "tox<4", | ||
| ], |
There was a problem hiding this comment.
FYI: pytest-freezegun got superseded by pytest-freezer.
| [testenv] | ||
| deps = -r{toxinidir}/test-requirements.txt | ||
| deps = -e{toxinidir}[testing] |
There was a problem hiding this comment.
I haven't tested invoking tox, but the change looks reasonable.
| @@ -60,8 +61,11 @@ | |||
| ], | |||
| extras_require={ | |||
| "testing": [ | |||
| "tox==3.14.2", | |||
| "pytest-freezegun==0.4.2", | |||
| "pytest<9", | |||
There was a problem hiding this comment.
I think in the past we tried to use == if possible instead of <.
I don't have a strong opinion about that. @Taliik What do you think about it?
There was a problem hiding this comment.
I guess it will be reasonable for stable requirements to have a more lenient version pinning, specifically in the "testing" section. That also reduces Dependabot spam. Let me know if you disagree.
There was a problem hiding this comment.
I'm fine with more lenient version pinning for testing 👍
There was a problem hiding this comment.
Thanks. Will you acknowledge?
About
A few adjustments to prepare unlocking Python 3.12 and eventually 3.13.
The patch mainly phases out use of
pkg_resourcesanddistutils.util.