-
Notifications
You must be signed in to change notification settings - Fork 97
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
Bump dev version #778
Bump dev version #778
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
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 think the version is off?
@@ -4,7 +4,7 @@ build-backend = "hatchling.build" | |||
|
|||
[project] | |||
name = "metricflow" | |||
version = "0.202.0" | |||
version = "0.202.0.dev4" |
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.
version = "0.202.0.dev4" | |
version = "0.203.0.dev1" |
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.
Ah that was not my understanding of how we move to dev versions. I was confused to see we had moved from 0.202.0.dev3 to 0.202.0, because it seems like a reversion? Is that how we've always done it and I'm just being crazy?
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.
Also, don't we start with dev0?
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 just went to check commit history and at least in recent bumps (excluding the last one you bumped to 0.202.0) we have been following the pattern of 0.200.0 -> 0.200.0.dev0 -> 0.201.0
. There seems have been some fluctuation over time about what pattern we use.
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.
Oops, yes, it should be 0.203.0.dev0
although I'm not too concerned with the dev version value itself, as long as nobody ever goes backwards.
We've actually always followed the <existing release version> -> <new version with pre-release> -> <new version>
standard. We moved from 0.140.0 to the 0.200.0.dev0 pre-release and then moved from the 0.200.0 pre-release to 0.200.0
Among the confusing things in our history we have:
- Devon's PR for the 0.202.0.dev0 pre-release has the wrong title, which I'll fix, although the git commit history might just be busted and I don't want to rewrite it just for this
- There was no pre-release for 0.201.0 - that was a straight release of the new version.
All of this confusion (and you're not the only one, it happened to Devon and to tell you the truth I need to look it up every time myself) tells me I need to move our version updating to be something scriptable, add that to the goofy shell script I wrote for making release PRs, and maybe put that up for review so all of us can use it for at least these mechanical dev version PRs.
Resolves #SL-978
Description
Bump dev version so we can do a dev release to get the new DATE PART feature into our APIs.