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

wheels not published to pypi, aarch64 missing #1328

Closed
jswhit opened this issue Jun 13, 2024 · 4 comments · Fixed by #1333
Closed

wheels not published to pypi, aarch64 missing #1328

jswhit opened this issue Jun 13, 2024 · 4 comments · Fixed by #1333

Comments

@jswhit
Copy link
Collaborator

jswhit commented Jun 13, 2024

Not sure why the wheels were not published to pypi (see https://github.com/Unidata/netcdf4-python/actions/runs/9506104839/workflow).

aarch64 is commented out in the workflow - can this be enabled? (we have been providing aarch64 wheels with multibuild)

@ocefpaf

@ocefpaf
Copy link
Collaborator

ocefpaf commented Jun 14, 2024

I commented it out during testing iteration, b/c it was super slow, and forgot to put is back. Trying it now in #1333

@ocefpaf
Copy link
Collaborator

ocefpaf commented Jun 14, 2024

Regarding the wheels, someone with PyPI admin access must set the token up. Or maybe you may want to try a new way for this with https://docs.pypi.org/trusted-publishers/. Apologies that I forgot to mentioned that before.

BTW, if you choose the former, the process should be similar to what happened in cftime. We need to create the PYPI token, and then add it as a GH secret in this repo, with the same names used here.

@dopplershift
Copy link
Member

Would wholly recommend using the trusted publisher stuff. Was easy to set up and have had no issues on MetPy.

@ocefpaf
Copy link
Collaborator

ocefpaf commented Jun 14, 2024

Would wholly recommend using the trusted publisher stuff. Was easy to set up and have had no issues on MetPy.

It is safer too BTW.

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 a pull request may close this issue.

3 participants