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
Fix version issues in the publish workflow #376
Fix version issues in the publish workflow #376
Conversation
Codecov Report
@@ Coverage Diff @@
## master #376 +/- ##
=======================================
Coverage 90.27% 90.27%
=======================================
Files 54 54
Lines 2386 2386
=======================================
Hits 2154 2154
Misses 232 232
Continue to review full report at Codecov.
|
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.
This looks like a sensible fix, just so I have it straight: the versions get updated according to the tag, if the GH release has a semver title, then required all the other files get updated, commited, and the tag gets moved forward, which has to be a force push. Right?
Is there any way we can change the push to git push --force --tags <the_tag_in_question>
instead of just git push --force
(if that is what is actually happening)? I must admit all these 3rd party actions make it quite hard to follow without digging into them...
Yeah, it already did that. This fix addresses:
Edit: |
Gotcha, tag me when you're done and I can review when I wake up. I stuck a related Q about building wheels on slack |
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.
This LGTM now, I'm sure we'll be doing another release soon so we can test it. I'll give @CasperWA a chance to look too.
This fixes the versioning issue in the publish workflow due to having it stored in
__init__.py
now.