Bump pytest to 9.0.3, pytest-cov to 6.1.1, hypothesis to 6.131.15#19172
Bump pytest to 9.0.3, pytest-cov to 6.1.1, hypothesis to 6.131.15#19172kirklandsign wants to merge 1 commit intomainfrom
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/19172
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 5 New FailuresAs of commit 6566850 with merge base cb4e5ae ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@kirklandsign has exported this pull request. If you are a Meta employee, you can view the originating Diff in D102679505. |
This PR needs a
|
There was a problem hiding this comment.
Pull request overview
Updates ExecuTorch’s pinned Python test dependencies used by the CI Docker images to newer, mutually-compatible versions (pytest 9.x + matching pytest-cov, plus a newer Hypothesis) to modernize OSS CI and avoid stacking incompatible bumps.
Changes:
- Bump
pytestfrom 7.2.0 to 9.0.3 - Bump
pytest-covfrom 4.1.0 to 6.1.1 (to match newer pytest) - Bump
hypothesisfrom 6.84.2 to 6.131.15
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| pytest==9.0.3 | ||
| pytest-cov==6.1.1 | ||
| expecttest==0.1.6 | ||
| hypothesis==6.84.2 | ||
| hypothesis==6.131.15 |
There was a problem hiding this comment.
After bumping CI to pytest 9.x, the repo’s development requirements are now inconsistent: requirements-dev.txt still caps pytest<9.0. This makes it harder to reproduce CI failures locally and can lead to dependency resolver conflicts for contributors. Consider updating/removing that cap in the same PR (or explicitly linking a landed follow-up) so dev and CI environments stay aligned.
Summary:
Brings executorch's open-source CI test-dependency pins forward:
- pytest: 7.2.0 → 9.0.3
- pytest-cov: 4.1.0 → 6.1.1 (required for pytest 9.x compatibility;
pytest-cov 4.x predates pytest 8/9 support)
- hypothesis: 6.84.2 → 6.131.15
\`requirements-ci.txt\` is the pinned dependency set used by the CI Docker
images. The previous pins predated pytest 8 and 9 entirely; bumping in
one go avoids stacking incompatible Dependabot PRs.
Also drops the \`pytest<9.0\` upper bound in \`requirements-dev.txt\`. That
cap pre-dated pytest 8/9 support and now actively conflicts with the CI
pin — without lifting it, contributors hit pip resolver errors when
trying to reproduce CI failures locally.
___
overriding_review_checks_triggers_an_audit_and_retroactive_review
Oncall Short Name: ai_infra_mobile_platform
Differential Revision: D102679505
0024498 to
6566850
Compare
Summary:
Brings executorch's open-source CI test-dependency pins forward:
pytest-cov 4.x predates pytest 8/9 support)
`requirements-ci.txt` is the pinned dependency set used by the CI Docker
images. The previous pins predated pytest 8 and 9 entirely; bumping in
one go avoids stacking incompatible Dependabot PRs.
Also drops the `pytest<9.0` upper bound in `requirements-dev.txt`. That
cap pre-dated pytest 8/9 support and now actively conflicts with the CI
pin — without lifting it, contributors hit pip resolver errors when
trying to reproduce CI failures locally.
overriding_review_checks_triggers_an_audit_and_retroactive_review
Oncall Short Name: ai_infra_mobile_platform
Differential Revision: D102679505