-
Notifications
You must be signed in to change notification settings - Fork 5
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
Release 0.3.1 of PyTNG #108
Comments
Ill cut now? |
@hmacdope what's your release procedure? It should match the new deployment workflow |
New release procedure should be:
|
Normally, cut tag on main, pull, then push to PyPi from my own computer, I assume the new workflow makes it easier? |
yeah you don't need to do any of that anymore, wheel building should be automatic - assuming the workflow doesn't have any bugs (it'll probably fail the first time around knowing how these things go). |
Cutting directly on github will trigger or do i need to actually |
Yes, just doing this will work: git tag -m "release 0.3.1 of pytng" v0.3.1
git push --tags origin Then when CI passes you can just do a release using that new tag. |
Looks like the wheel upload went fine: https://test.pypi.org/project/pytng/#files Should be good to cut a release. |
Legendary, thanks so much. Good practice for me to understand the no-local workflow as well. |
Done and tested. |
@hmacdope you're probably going to need to update the conda-forge recipe quite a bit because of the numpy pins, iirc the current defaults aren't the same as what conda-forge expects. |
I don't think I'm a maintainer for that recipe but if you ping me for review I can add the necessary changes. |
Cheers, will do. Thanks for sorting this out I was very slow, my apologies. |
A release of pytng is necessary in order to allow for py3.12 support - this is blocking the core library here: MDAnalysis/mdanalysis#4327
The text was updated successfully, but these errors were encountered: