The Wayback Machine - https://web.archive.org/web/20240106173906/https://github.com/matplotlib/matplotlib/pull/27563
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

Enable PyPI publishing from GitHub Actions #27563

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Dec 23, 2023

PR summary

Also bump the artifacts actions to v4.

Note that normally the whole publish job should be skipped on non-release events, but I want to confirm that the artifacts download stuff works.

PR checklist

@QuLogic QuLogic added Maintenance CI: Run cibuildwheel Run wheel building tests on a PR labels Dec 23, 2023
@QuLogic QuLogic added this to the v3.9.0 milestone Dec 23, 2023
@QuLogic QuLogic marked this pull request as draft December 23, 2023 10:04
@QuLogic QuLogic force-pushed the pypi-publishing branch 2 times, most recently from 83bb4f8 to bfdd037 Compare January 4, 2024 04:17
@QuLogic
Copy link
Member Author

QuLogic commented Jan 4, 2024

We do successfully have all the wheels we'd like, I think:
https://github.com/matplotlib/matplotlib/actions/runs/7405461070/job/20149100577?pr=27563#step:4:5

I'm going to revert this back to only attempting to publish when there's an actual tag.

@QuLogic
Copy link
Member Author

QuLogic commented Jan 4, 2024

The remaining question is whether the nightly upload workflow will continue to work, i.e., whether the download procedure there is compatible with actions/artifacts@v4. Do you have any idea about this @matthewfeickert?

@matthewfeickert
Copy link
Contributor

The remaining question is whether the nightly upload workflow will continue to work, i.e., whether the download procedure there is compatible with actions/artifacts@v4.

@QuLogic Sadly I think the answer is no, without refactors. On 2023-12-15 @henryiii alerted the IRIS-HEP Slack to this with the following

PSA: make sure you know the consequences before you merge upload/download-artifact’s v4 updates. Upload artifact no longer merges artifacts automatically, which is a massive change and breaks pretty much all cibuildwheel or coverage workflows. Also websites with pyodide workflows. Etc.

Actually though, you may be good as @henryiii is awesome and updated sp-repo-review to be able to automatically check for this and https://learn.scientific-python.org/development/guides/repo-review/?repo=matplotlib%2Fmatplotlib&branch=main shows that GH104

GH104 Use unique names for upload-artifact ✅

is checked as working! However, things broke for pyhf and we also were checked off as passing. :/ (haven't had time to debug this yet, so might be different)

I have yet to go through and fix things for my own projects yet (I've been AFK on holiday and my only laptop died so I have only recently gotten a loaner that I can use) so once I do that (probably next week) I can open a PR for matplotlib that would migrate the actions to v4 but if you don't want to take the time to debug that now then I would suggest staying on v3 and then updating to v4 in a new PR.

with:
name: sdist
name: cibw-sdist
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah @QuLogic sorry I didn't review before commenting, but given that you've given distinct names I think everything is fine as you're doing what @henryiii did in https://github.com/scientific-python/cookie/pull/350/files. So I think this is good (without having done it myself). 👍

@henryiii
Copy link

henryiii commented Jan 4, 2024

Pyhf seems to have updated one but not both actions. You have to have them match.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: Run cibuildwheel Run wheel building tests on a PR Maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants