-
Notifications
You must be signed in to change notification settings - Fork 46
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
Switching from versioneer to setuptools-scm #165
Conversation
I think you also need to remove this stuff Lines 1 to 4 in 744209f
plus this file: https://github.com/xgcm/xrft/blob/master/xrft/_version.py |
So you can see from the CI that the version is not getting set right. It's currently showing up as https://github.com/xgcm/xrft/pull/165/checks?check_run_id=3475121204#step:5:136 |
@roxyboy - did you see Tom's comments in #164 (comment)? |
Yes, and I've been trying to copy and paste files that he's edited but I have yet to figure it out :(( |
.github/workflows/release.yaml
Outdated
TWINE_USERNAME: "__token__" | ||
TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }} |
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.
Did you actually change the secrets on the repo? If not, this change will break the release process.
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.
Thanks for pointing this out. I didn't so... I'll revert to the previous edits.
Here are another thread on the same topic: pypa/setuptools-scm#414 From reading that, one fix could be
|
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 found more stuff that doesn't make sense here.
.github/workflows/pre-commit.yaml
Outdated
steps: | ||
- uses: actions/checkout@v2 | ||
with: | ||
fetch-depth: 0 |
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.
It doesn't matter if you add this in the pre-commit workflow. It needs to be added in the workflow where you care about the version number. What you do in this workflow won't affect the other workflows.
Edit: It is actually working! See https://github.com/xgcm/xrft/runs/3569028145?check_suite_focus=true#step:5:128 |
I really love to be pinged on a packaging issue that get resolved in less than 10 min until I get here 😄 Shameless plug though I'm keeping a "package skeleton" as an alternative to cookie cutters out there: https://github.com/ioos/ioos-python-package-skeleton It is supposed to be a copy-n-paste updated references for scientific Python packaging. I'm writing docs on it to make it easier to navigate them. |
Yes we know about the package skeleton. Unfortunately this project predates it. |
Should we make a new release so that PyPI is up to date? |
Yes, can you try to make the release? We released 0.3.1 on GitHub, but it didn't work for pypi (thus this PR): https://github.com/xgcm/xrft/releases Use |
I just ran |
Bravo!! 👯 |
A PR to switch from
versioneer
tosetuptools_scm
for managing versions.fixes #164