Skip to content

Run the Pydantic v1 test in an overlay and un-skip the gevent test - #414

Open
DABH wants to merge 1 commit into
mainfrom
ci-cleanup
Open

DABH wants to merge 1 commit into
mainfrom
ci-cleanup

Conversation

@DABH

@DABH DABH commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

This PR fixes two things in the root build-lint-test job.

The step named "Uninstall pydantic" is really the Pydantic v1 sample's test. It downgraded the shared virtualenv in place and had to stay last. It now runs that one test with uv run --with "pydantic>=1.10,<2", which layers Pydantic 1 over the project environment, and disables the langsmith pytest plugin, which needs Pydantic 2.

The "Gevent test" step was gated on Python 3.12, which is not in the matrix, so it never ran. It now runs on 3.14.

@DABH
DABH requested a review from a team as a code owner November 12, 2026 20:33
The old step uninstalled Pydantic 2 from the shared virtualenv and had to
stay last; uv can layer Pydantic 1 over the project environment for that
one test instead. The gevent step was gated on Python 3.12, which is not
in the matrix, so it never ran.
@DABH
DABH enabled auto-merge (squash) November 13, 2026 19:27
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