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

Bump required pytest version to 4.6 #98

Merged
merged 1 commit into from
Oct 5, 2024
Merged

Conversation

diazona
Copy link
Contributor

@diazona diazona commented Oct 5, 2024

We have started getting an error with pytest>=4,<4.6 when setuptools>=71 (or typeguard>=4.2) is also installed, due to a conflict with older pytest and setuptools' vendored version of typeguard. Out of various ways to resolve this conflict, this is the only one that doesn't involve putting an upper bound on the version of some package.

It should still be technically possible to use older versions of pytest along with setuptools<71, if someone really needs to do that, but there isn't any way to express that dependency constraint, and I'm not going to go to any trouble to support long-obsolete versions of packages.

Testing notes

  • If I add typeguard<4.2 to the test requirements in tox.ini, then I run into Failure under pytest-6.2.5 agronholm/typeguard#343
  • If I add setuptools<71 then everything works (since it doesn't require any version of typeguard at all)
  • If I change to requiring pytest>=6.2, then again everything works
  • It turns out I can go as far back as pytest>=4.6 and still everything works; it looks like there was a change to how and when plugins are loaded in 4.6 that affects this

Hence I'm bumping to 4.6 in this PR.

Closes #97

We have started getting an error with pytest>=4,<4.6 when setuptools>=71
(or typeguard>=4.2) is also installed, due to a conflict with older
pytest and setuptools' vendored version of typeguard. Out of various
ways to resolve this conflict, this is the only one that doesn't involve
putting an upper bound on the version of some package.

It should still be technically possible to use older versions of pytest
along with setuptools<71, if someone really needs to do that, but there
isn't any way to express that dependency constraint, and I'm not going
to go to any trouble to support long-obsolete versions of packages.
@diazona
Copy link
Contributor Author

diazona commented Oct 5, 2024

P.S. I'll make a separate PR after this one to support pytest 8.

@diazona diazona merged commit 39cad5b into master Oct 5, 2024
55 checks passed
@diazona diazona deleted the bump-pytest-to-4.6/1/dev branch October 5, 2024 22:19
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.

Tests fail on pytest 3.8 and 3.9 with pytest 4
1 participant