-
Notifications
You must be signed in to change notification settings - Fork 504
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
Revive the changelog #300
Revive the changelog #300
Conversation
Signed-off-by: Joe Farro <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #300 +/- ##
=======================================
Coverage 82.57% 82.57%
=======================================
Files 140 140
Lines 3151 3151
Branches 651 651
=======================================
Hits 2602 2602
Misses 438 438
Partials 111 111 Continue to review full report at Codecov.
|
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.
What is the distinction between Next
and Changes merged into master
? Seems like the 4 PRs currently under Next
are all also part of master but not listed under Changes merged into master
@everett980 Historically, we haven't used semantic versioning with the UI because we have had only minimal API contracts. We just merged everything to master and the It seems prudent to start using semantic versioning to indicate whether or not changes have any impact on our API contract(s). With this approach, a set of changes merged to master but not yet characterized by a semantic version can be considered unreleased. The |
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.
Looks good, once we have done a few releases we'll probably want a (fairly short) document on the release process.
Signed-off-by: Joe Farro <[email protected]>
Signed-off-by: Joe Farro <[email protected]>
* Revive the changelog Signed-off-by: Joe Farro <[email protected]> * Add helper script to generate CHANGELOG entries Signed-off-by: Joe Farro <[email protected]> * One more changelog item, fix date typo Signed-off-by: Joe Farro <[email protected]> * Fix phrasing in changelog Signed-off-by: Joe Farro <[email protected]> Signed-off-by: Everett Ross <[email protected]>
* Revive the changelog Signed-off-by: Joe Farro <[email protected]> * Add helper script to generate CHANGELOG entries Signed-off-by: Joe Farro <[email protected]> * One more changelog item, fix date typo Signed-off-by: Joe Farro <[email protected]> * Fix phrasing in changelog Signed-off-by: Joe Farro <[email protected]> Signed-off-by: vvvprabhakar <[email protected]>
* Revive the changelog Signed-off-by: Joe Farro <[email protected]> * Add helper script to generate CHANGELOG entries Signed-off-by: Joe Farro <[email protected]> * One more changelog item, fix date typo Signed-off-by: Joe Farro <[email protected]> * Fix phrasing in changelog Signed-off-by: Joe Farro <[email protected]> Signed-off-by: Everett Ross <[email protected]> Signed-off-by: vvvprabhakar <[email protected]>
Which problem is this PR solving?
The changelog is out of date.
Short description of the changes
Bringing the changelog current.
Also, this switches to a format where we formally bless
master
as release ready by cutting a release in GitHub.