-
Notifications
You must be signed in to change notification settings - Fork 132
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
Use dynamic versioning to single-source version from git tags #415
Conversation
Not sure what these failing unit tests are .... |
Coverage after merging jb/dynamic-version into master will be
Coverage Report for Changed Files
|
a2ea73c
to
fb05d3b
Compare
Coverage after merging jb/dynamic-version into master will be
Coverage Report for Changed Files
|
OK, figured out it was because Github Action workflows only do a shallow fetch; but the "single source" of version with the PR becomes the git history ... I've tried a couple things to just pull the git tags, but they all failed. Checked out a some other repositories that deploy via Github actions (prefect, pandas), and looks like they all do a full fetch. |
Coverage after merging jb/dynamic-version into master will be
Coverage Report for Changed Files
|
Oh, this is slightly different from what I was thinking. I simply wanted to define the version in either Lines 1 to 3 in 121314a
This is cool tho. I do really like the But maybe I just need to get used to this :) What do other folks think? @isaacbrodsky @nrabinowitz @dfellis |
I see. The only way I can think of for having a The main benefit of using tags, imo, is that you can cut a release without having that one odd PR just to bump the version in-code 😅 (e.g., via Github's Release UI)—but I can also see how that might not be necessarily desirable. |
We'd still usually need one for the changelog, I think. |
Thanks for looking into this, but like I said above, I think I got most of what I wanted working previously, and I don't mind the extra PR for a release. |
Closing for now, but maybe we'll revisit in the future. |
Does this address #383 ?
Example local install:
and then with a tag:
Currently using the default versioning scheme; but configurable.
Wasn't sure what didn't work last time (in #378)