-
Notifications
You must be signed in to change notification settings - Fork 14
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 #297
Bump dev version #297
Conversation
pyproject.toml
Outdated
@@ -1,6 +1,6 @@ | |||
[project] | |||
name = "dbt-semantic-interfaces" | |||
version = "0.6.0" | |||
version = "0.6.1dev0" |
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 should be 0.6.1.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.
Actually, should this be 0.6.1.dev0 or should it be 0.7.0.dev0? We're on new feature kind of stuff, right? Whichever works, though.
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.
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 don't think we should bump to 0.7.0 unless it's a breaking change. If we do, it will require another round of bumping version requirements in core which should not be necessary for non-breaking changes.
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.
It looks like the DSI and MF docs differ on the naming practices so that's why it feels like there's thrash here. https://www.notion.so/dbtlabs/Metricflow-Build-Release-Process-81023685d93a45da8211b05dc36b7bf8?pvs=4#9accf24d2929404ba7b9337ad21a33e3
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.
Updated to suit PEP!
Description
Needed to get some new fields into MF (specifically
cumulative_type_params
).