Skip to content

Make the numpy and gymnasium ignores actually work, and add pytest - #31

Open
DenisDrobyshev wants to merge 2 commits into
mainfrom
ignores-that-work
Open

Make the numpy and gymnasium ignores actually work, and add pytest#31
DenisDrobyshev wants to merge 2 commits into
mainfrom
ignores-that-work

Conversation

@DenisDrobyshev

Copy link
Copy Markdown
Member

My own change in #23 left numpy and gymnasium on update-types: ["version-update:semver-major"], which does not do what it looks like it does.

#29 raised numpy from >=1.21 to >=2.2.6 and walked straight past that filter — a two-major jump. For a >= constraint Dependabot files a requirement update rather than a version update, so update-types never matches. That behaviour is already written into lemma's dependabot.yml; I copied half the lesson and left these two entries as they were.

Now unconditional, like torch beside them.

pytest

Joins mypy for the other reason. pytest 9 requires Python ≥3.10 and mypy 2.3 requires ≥3.11, so both fail the 3.9 leg at install — #17 and #25, closed today, 41s and 44s respectively while every other leg passed.

Ignoring them is a holding action rather than an answer. #30 is about the constraint actually causing it: requires-python = ">=3.9", with 3.9 unsupported upstream since October 2025 and the toolchain frozen behind it. That is a user-facing support decision, so it is an issue rather than a pull request.

My own change in #23 left these two on update-types: ["version-update:
semver-major"], which does not do what it looks like it does. #29 raised numpy
from >=1.21 to >=2.2.6 and walked straight past it, because for a `>=`
constraint Dependabot files a "requirement update" rather than a version
update - the behaviour already written into lemma's configuration and which I
copied only half of.

pytest joins mypy for the other reason: pytest 9 requires Python >=3.10 and
mypy 2.3 requires >=3.11, so both fail the 3.9 leg at install. Ignoring them is
a holding action, not an answer - #30 is about the support matrix that is
actually causing this.
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.

1 participant