Skip to content

Conversation

@gmloose
Copy link
Collaborator

@gmloose gmloose commented Apr 23, 2025

The current GitHub action that builds and publishes a release, fails to upload the source tar ball. The MR fixes the broken action. The probable cause of this was probably a half-baked upgrade to v4 of the upload and download actions.

@gmloose gmloose self-assigned this Apr 23, 2025
@aroffringa
Copy link
Contributor

Just a side comment: It might be good to use a descriptive title for the PR ; having just a number in the commit log isn't as easy. Also, if you mention the number not in the title but in the body of the PR, there will be a msg in ticket #276 that it has been mentioned here.

@gmloose
Copy link
Collaborator Author

gmloose commented Apr 24, 2025

Sorry, I meant to update the description before removing the Draft status. I hadn't realized it was in its Draft status already causing confusion. 😞

@gmloose gmloose changed the title Fix issue #276 Fix GitHub action that uploads to PyPI Apr 24, 2025
gmloose added 2 commits April 24, 2025 17:09
* Make sure we publish on TestPyPI

* Remove Ubuntu 20.04 from workflow

* Build wheels for all Python versions

* Use latest version of MacOS

* Revert to publishing on PyPI

* Revert "Revert to publishing on PyPI"

This reverts commit bd1e0a2.

* Revert "Build wheels for all Python versions"

This reverts commit 700e712.

* Run Linux workflow alsowhen tags are created

* Switch to MacOS-13 to force Intel-based build

* Reapply "Revert to publishing on PyPI"

This reverts commit 64c39a6.
@gmloose gmloose marked this pull request as ready for review April 24, 2025 15:14
@gmloose gmloose requested a review from tammojan April 25, 2025 10:21
@tammojan
Copy link
Contributor

Looks good to me! Unfortunately the macos build fails on some numpy-thing, but I think that's unrelated.

@gmloose
Copy link
Collaborator Author

gmloose commented Apr 25, 2025

Looks good to me! Unfortunately the macos build fails on some numpy-thing, but I think that's unrelated.

Indeed. This issue has already been reported: casacore/homebrew-tap#11

@gmloose gmloose merged commit 8fcf796 into master Apr 25, 2025
5 of 6 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.

4 participants