FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci: Pin transitive dependencies for tests on Python>=3.8 by alexander-alderman-webb · Pull Request #6437 · getsentry/sentry-python · GitHub

ci: Pin transitive dependencies for tests on Python>=3.8 - #6437

Merged
alexander-alderman-webb merged 30 commits into
masterfrom
webb/populate-tox/transitive-dependencies
Jun 2, 2026
Merged

ci: Pin transitive dependencies for tests on Python>=3.8#6437
alexander-alderman-webb merged 30 commits into
masterfrom
webb/populate-tox/transitive-dependencies

Conversation

alexander-alderman-webb commented May 28, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Add transitive dependencies to all test groups that are generated by populate_tox.py.
Dependency versions are resolved via a dry run of pip through uv on the relevant Python version.
Pins one set of dependencies per Python version and per library version.

Update DEPENDENCIES_CACHE to also be keyed on the Python version, since dependency versions of a package depend on the Python version. The cache is now indexed on the package name, package version, and Python version.

Issues

Reminders

github-actions Bot commented May 28, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Codecov Results 📊

88336 passed | ⏭️ 6007 skipped | Total: 94343 | Pass Rate: 93.63% | Execution Time: 302m 5s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +161
Passed Tests 📈 +163
Failed Tests
Skipped Tests 📉 -2

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2311 uncovered lines.
❌ Project coverage is 89.97%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.98%    89.97%    -0.01%
==========================================
  Files          191       191         —
  Lines        23064     23040       -24
  Branches      7922      7908       -14
==========================================
+ Hits         20752     20729       -23
- Misses        2312      2311        -1
- Partials      1310      1308        -2

Generated by Codecov Action

alexander-alderman-webb marked this pull request as ready for review May 28, 2026 13:30
alexander-alderman-webb requested a review from a team as a code owner May 28, 2026 13:30
alexander-alderman-webb changed the title ci: Pin transitive dependencies for tests suites ci: Pin transitive dependencies for tests suites ran on Python>=3.8 May 28, 2026
alexander-alderman-webb changed the title ci: Pin transitive dependencies for tests suites ran on Python>=3.8 ci: Pin transitive dependencies for tests suites on Python>=3.8 May 28, 2026
alexander-alderman-webb marked this pull request as draft May 28, 2026 13:37
Comment thread scripts/populate_tox/populate_tox.py Outdated
Comment thread scripts/populate_tox/populate_tox.py Outdated
Comment thread scripts/populate_tox/populate_tox.py Outdated
alexander-alderman-webb changed the base branch from master to webb/setuptools June 1, 2026 11:23
alexander-alderman-webb changed the title ci: Pin transitive dependencies for tests suites on Python>=3.8 ci: Pin transitive dependencies for tests on Python>=3.8 Jun 1, 2026
alexander-alderman-webb marked this pull request as ready for review June 1, 2026 11:30
Base automatically changed from webb/setuptools to master June 1, 2026 11:37

cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Cursor Bugbot has reviewed your changes and found 1 potential issue.

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d26dd8a. Configure here.

sl0thentr0py left a comment
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

maybe time to add tox.ini to .gitattributes as well

sentrivana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM overall, but -latest should ideally be a verbatim copy of -v{max}, including the transitive pins. Otherwise they're two different test targets, and -latest succeeding locally when developing doesn't guarantee that CI will pass on the corresponding latest version. (That's never a guarantee, but I wouldn't introduce additional drift.)

alexander-alderman-webb merged commit d7a1cc4 into master Jun 2, 2026
270 of 272 checks passed
alexander-alderman-webb deleted the webb/populate-tox/transitive-dependencies branch June 2, 2026 09:13
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
)

Add transitive dependency pins to test groups generated by `populate_tox.py`.
Dependency versions are resolved through uv using a pip dry run on the relevant Python version, producing pins per Python version and library version.
Key `DEPENDENCIES_CACHE` by package name, package version, and Python version, since resolved dependency versions vary across Python versions.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL