Skip to content

ci: Remove asttokens from requirements-testing.txt - #6615

Merged
alexander-alderman-webb merged 6 commits into
masterfrom
webb/requirements-testing/asttokens
Jun 26, 2026
Merged

ci: Remove asttokens from requirements-testing.txt#6615
alexander-alderman-webb merged 6 commits into
masterfrom
webb/requirements-testing/asttokens

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Description

Remove the package from requirements-testing.txt and add the package as a dependency to only the test suites that rely on the package (if applicable).

This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.

Issues

Reminders

@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

90151 passed | ⏭️ 6305 skipped | Total: 96456 | Pass Rate: 93.46% | Execution Time: 317m 30s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2397 uncovered lines.
✅ Project coverage is 89.93%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.90%    89.93%    +0.03%
==========================================
  Files          192       192         —
  Lines        23809     23809         —
  Branches      8218      8218         —
==========================================
+ Hits         21404     21412        +8
- Misses        2405      2397        -8
- Partials      1341      1339        -2

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review June 23, 2026 09:22
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner June 23, 2026 09:22

@ericapisani ericapisani left a comment

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.

LGTM

Comment thread scripts/populate_tox/config.py Outdated
@alexander-alderman-webb
alexander-alderman-webb enabled auto-merge (squash) June 26, 2026 13:28
@alexander-alderman-webb
alexander-alderman-webb merged commit 9c02b35 into master Jun 26, 2026
144 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/requirements-testing/asttokens branch June 26, 2026 13:28
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
Remove the package from `requirements-testing.txt` and add it only as a dependency of the test suites that rely on it.
This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants