Skip to content

test(tox): Unpin pytest for Python 3.8+ common tests #3697

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

szokeasaurusrex
Copy link
Member

@szokeasaurusrex szokeasaurusrex commented Oct 28, 2024

This pin appears to be unnecessary on Python 3.8+.

ref #3035

Copy link

codecov bot commented Oct 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.30%. Comparing base (1ce7c31) to head (070ba93).
Report is 1 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3697      +/-   ##
==========================================
- Coverage   84.35%   84.30%   -0.05%     
==========================================
  Files         133      133              
  Lines       14257    14257              
  Branches     2405     2405              
==========================================
- Hits        12026    12019       -7     
- Misses       1482     1489       +7     
  Partials      749      749              

see 4 files with indirect coverage changes

@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/unpin-pytest-common branch from 56c1dc8 to 92ad1d5 Compare October 28, 2024 11:55
@szokeasaurusrex szokeasaurusrex changed the title ci(tox): Remove pytest pin for common tests ci(tox): Unpin pytest for Python 3.8+ common tests Oct 28, 2024
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review October 28, 2024 12:35
Base automatically changed from szokeasaurusrex/test-unpin-pytest-no-test-change to master October 28, 2024 12:36
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/unpin-pytest-common branch from 92ad1d5 to 070ba93 Compare October 28, 2024 13:13
@szokeasaurusrex szokeasaurusrex changed the title ci(tox): Unpin pytest for Python 3.8+ common tests test(tox): Unpin pytest for Python 3.8+ common tests Oct 28, 2024
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) October 28, 2024 13:27
@szokeasaurusrex szokeasaurusrex merged commit 200be87 into master Oct 28, 2024
136 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/unpin-pytest-common branch October 28, 2024 13:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants