-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
R4R: Simplify version handling, rely on git describe #2318
Conversation
I think this is an excellent idea but cc @greg-szabo are we still parsing |
We do, but we don't build automated gaiad builds anymore. When we decide to do it again, we'll have to rewrite/fix the automation. Feel free to go ahead with this change. Historically (I mean in the Tendermint repo) we only tagged the code when a new release was cut. I guess with moving the version information into tags, you want to change that process? (I couldn't make the team immediately update the version number on develop when a new release was deployed.) In Tendermint, the automated release also creates a tag on master, I wonder if that becomes available in develop too. |
@greg-szabo dixit:
No, we don't want to. This is all about providing develop builds with numeric versioning schema |
The change I'm thinking of is that when you deploy a release to master, you have to add a new tag to develop at least... (So the master deployments version number is the tag that was created on develop during the previous release - or something newer than that). |
4c5cc9c
to
13c67be
Compare
Codecov Report
@@ Coverage Diff @@
## develop #2318 +/- ##
==========================================
- Coverage 64.97% 63.9% -1.08%
==========================================
Files 135 140 +5
Lines 8418 8714 +296
==========================================
+ Hits 5470 5569 +99
- Misses 2587 2766 +179
- Partials 361 379 +18 |
Odd CI failures, possibly unrelated to this PR? |
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.
Tested ACK
Simplify version handling, rely on git describe
Closes: #1941
docs/
)PENDING.md
with issue #Files changed
in the github PR explorerFor Admin Use: