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

Add wheel with a C extension to test mounting #229

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

stewartmiles
Copy link
Contributor

@stewartmiles stewartmiles commented Sep 20, 2024

This recreates the source to the tests/minimext*.whl wheels with a couple of differences:

  • The C extension is placed in the minimext package rather than being a top level package.
  • A Python implementation of the C extension is provided to make it easier for pure Python programmers to understand / modify.

This also adds a script build_wheels.sh that will install Python interpreters and build the minimext wheel for different Python versions with and without adding the EXTENSIONS metadata file.

Fixes #227
Related to #222

This recreates the source to the `tests/minimext*.whl` wheels
with a couple of differences:
* The C extension is placed in the minimext package rather than being
  a top level package.
* A Python implementation of the C extension is provided to make it
  easier for pure Python programmers to understand / modify.

This also adds a script `build_wheels.sh` that will install Python
interpreters and build the minimext wheel for different Python
versions with and without adding the `EXTENSIONS` metadata file.

Related to pypa#222
Copy link

codecov bot commented Sep 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.50%. Comparing base (674a491) to head (2f0b158).
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #229   +/-   ##
=======================================
  Coverage   81.49%   81.50%           
=======================================
  Files          24       24           
  Lines        8885     8888    +3     
  Branches     1747     1746    -1     
=======================================
+ Hits         7241     7244    +3     
- Misses       1300     1301    +1     
+ Partials      344      343    -1     
Flag Coverage Δ
unittests 80.60% <ø> (-0.02%) ⬇️

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.

@stewartmiles
Copy link
Contributor Author

Similar to other commits, failures are due to the following job failing on macOS:

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v3
with:
flags: unittests
files: coverage.xml

@vsajip vsajip merged commit 6fbadf1 into pypa:master Oct 7, 2024
47 of 64 checks passed
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.

Add sources tests/minimext-0.1-*.whl wheels
2 participants