-
Notifications
You must be signed in to change notification settings - Fork 310
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
[DOCS] update changelog for 0.24.0 #2052
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2052 +/- ##
============================================
+ Coverage 78.83% 81.07% +2.24%
+ Complexity 1015 827 -188
============================================
Files 199 165 -34
Lines 5557 4799 -758
Branches 421 335 -86
============================================
- Hits 4381 3891 -490
+ Misses 726 584 -142
+ Partials 450 324 -126
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
CHANGELOG.md
Outdated
## [Unreleased](https://github.com/MarquezProject/marquez/compare/0.23.0...HEAD) | ||
## [Unreleased](https://github.com/MarquezProject/marquez/compare/0.24.0...HEAD) | ||
|
||
## [0.24.0](https://github.com/MarquezProject/marquez/compare/0.23.0...0.24.0) - 2022-07-28 |
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.
@merobi-hub: We would only update the changelog for 0.24.0
right before the release. The Unreleased
section can still contain a list of changes (that you've added in this PR) that have yet to be released without an official release.
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.
@wslulciuc Thanks. My thinking was that if I opened a PR ahead of time the changelog could be reviewed and revised ahead of the release if needed without losing time
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.
Ahh makes sense! I didn't want to merge the PR until the tag / release was out, but looks like we're scheduled for the release today, so let's get this merged!
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.
@merobi-hub We'll also want to add a recent fix by @collado-mike (see 2053
)
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.
@wslulciuc PR 2053 has been added. Good to go?
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.
LGTM, but you'll also want to add a recent fix by @collado-mike (see 2053
)
@wslulciuc 2053 has been added. Are we good to go? |
@wslulciuc wait, I should update the date |
Signed-off-by: Michael Robinson <merobi@gmail.com>
e927fdd
to
72dac25
Compare
Signed-off-by: Michael Robinson merobi@gmail.com
Problem
Changelog needs to be updated for 0.24.0.
Solution
Updates changelog for release.
Checklist
CHANGELOG.md
with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary).sql
database schema migration according to Flyway's naming convention (if relevant)