-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
PyPI Publish GH Action #400
Conversation
Initial nonworking draft of structure
add link to cibuildwheel
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #400 +/- ##
==========================================
+ Coverage 95.84% 95.91% +0.06%
==========================================
Files 24 24
Lines 7275 7312 +37
==========================================
+ Hits 6973 7013 +40
+ Misses 302 299 -3 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you change the branch I think the PR should be merged and we can start getting it to work. Unfortunately, I don't know much about this so I can't provide very useful feedback.
- uses: actions/checkout@v4 | ||
|
||
- name: Build wheels | ||
uses: pypa/[email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does cibuildwheel
need to be installed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could be wrong, but not to my knowledge. uses: pypa/[email protected]
should install cibuildwheel
within the GH action runner.
runs-on: ubuntu-latest | ||
environment: | ||
name: pypi | ||
url: https://pypi.org/p/build123d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is https://pypi.org/p/build123d
short for https://pypi.org/project/build123d
? Do either work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that is a shortcut and either will work (from PyPI docs)
This is ready for testing, it is setup to publish to the PyPI testing target (not any real PyPI target). There may need to be some changes made to
pyproject.toml
but I have not done that yet. Further info aboutpyproject.toml
is here:https://cibuildwheel.readthedocs.io/en/stable/options/