Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Misc fixes #449

Merged
merged 5 commits into from
Oct 30, 2023
Merged

Misc fixes #449

merged 5 commits into from
Oct 30, 2023

Conversation

skirpichev
Copy link
Contributor

@skirpichev skirpichev commented Oct 30, 2023

9fce264 Don't use private API: _Py_HashPointer
1de1bc2 Add workaround for shutil.copytree on CPython 3.7
2b805ac Run Cython tests in coverage job
b723d71 Run _set_default_context() fixture per function
ac2cfe2 Add pytest_report_header(), like test/runtests.py

@codecov-commenter
Copy link

codecov-commenter commented Oct 30, 2023

Codecov Report

Merging #449 (ac2cfe2) into master (1253106) will increase coverage by 0.13%.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master     #449      +/-   ##
==========================================
+ Coverage   84.83%   84.97%   +0.13%     
==========================================
  Files          49       49              
  Lines       11738    11738              
  Branches     2206     2206              
==========================================
+ Hits         9958     9974      +16     
+ Misses       1780     1764      -16     
Files Coverage Δ
src/gmpy2_hash.c 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@casevh
Copy link
Collaborator

casevh commented Oct 30, 2023

Thanks again for all the cleanup work.

@casevh casevh merged commit 1256479 into aleaxit:master Oct 30, 2023
9 checks passed
@skirpichev skirpichev deleted the misc branch October 30, 2023 14:19
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