-
Notifications
You must be signed in to change notification settings - Fork 414
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
git describe gives incorrect output #4891
Comments
The thing is that we often synchronise the bugfix between the branches and main. So if we were tagging 2.9+alpha when branching 2.9, then something that would morally be 2.9.4 + dev towards 3.0 might be reported as 2.9+alpha, which doesn't seem right either. I feel like if we want to be accurate there, we have to maintain the version explicitly somewhere in a file. |
Would it help if there were a way to use something similar to Related: #3856 |
I suppose, but why? It feels like a lot of effort to stick to |
Oh, in that case we would also tag Sorry I wasn't clear, you tag the alpha version when you branch for the stable, but you tag the next version. So in this case, the first commit of 3.0, would be tagged Using a file is also very fine I think. |
I was splitting hair TBH. I'm usually not a big fan of extra tags as these just accumulate and I find a big list of tags with a lot of alphas and betas a bit messy, but it's not a good argument. @jberdine, thinking about you said again, we could also teach opam about the concept of "alpha" and "beta" rather than introduce new character meanings. At the end of the day, alpha and beta versions are widely used concepts. |
I also don't like noise in tags, but that's one extra tag for each major release so I can live with it. |
These commits seem good to me. |
Fixed, thanks to all. |
In
main
:That's annoying if one wants to use
dune --version
with dev builds.I'd suggest tagging
3.0+alpha
etc... when branching. In this case, we'd miss a 2.9+alpha and 3.0+alpha tag.The text was updated successfully, but these errors were encountered: