Skip to content

Conversation

bdraco
Copy link
Member

@bdraco bdraco commented Sep 10, 2024

#1005 (comment)
Remove pre Python 3.8+ compat code from tests

@bdraco bdraco added the bot:chronographer:skip This PR does not need to include a change note label Sep 10, 2024
@bdraco bdraco changed the title Remove pre Python 3.8+ compat code from tests Remove compat code for Python older than 3.8 from tests Sep 10, 2024
@bdraco bdraco marked this pull request as ready for review September 10, 2024 01:35
@bdraco bdraco requested a review from asvetlov as a code owner September 10, 2024 01:35
Copy link

codecov bot commented Sep 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.26%. Comparing base (6197e51) to head (cabed5a).
Report is 23 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1010      +/-   ##
==========================================
+ Coverage   94.19%   94.26%   +0.07%     
==========================================
  Files          22       22              
  Lines        2430     2425       -5     
  Branches      278      278              
==========================================
- Hits         2289     2286       -3     
+ Misses         69       67       -2     
  Partials       72       72              
Flag Coverage Δ
CI-GHA 94.26% <100.00%> (+0.07%) ⬆️
MyPy 67.14% <100.00%> (+0.02%) ⬆️
pytest 97.87% <100.00%> (+0.18%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bdraco bdraco merged commit a0c04d5 into master Sep 10, 2024
44 of 47 checks passed
@bdraco bdraco deleted the missed_38_tests_redux branch September 10, 2024 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:skip This PR does not need to include a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants