tests: Move beam under toxgen#4759
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4759 +/- ##
==========================================
+ Coverage 84.84% 84.86% +0.01%
==========================================
Files 157 157
Lines 16138 16138
Branches 2744 2744
==========================================
+ Hits 13692 13695 +3
+ Misses 1656 1653 -3
Partials 790 790 |
There was a problem hiding this comment.
Bug: CI Matrix Update Breaks Python 3.8 Compatibility
Removing Python 3.8 from the CI matrix means several task integrations (beam, arq, celery, dramatiq, spark) with existing Python 3.8 tox configurations are no longer tested, creating a coverage gap. Also, the "Test beam latest" step fails because the workflow tries to run a beam-latest tox environment, but tox.ini now only defines specific versioned beam targets.
.github/workflows/test-integrations-tasks.yml#L31-L32
sentry-python/.github/workflows/test-integrations-tasks.yml
Lines 31 to 32 in bca6960
.github/workflows/test-integrations-tasks.yml#L66-L70
sentry-python/.github/workflows/test-integrations-tasks.yml
Lines 66 to 70 in bca6960
Moving a test suite under toxgen effectively means removing it from the |
Ref #4506