Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Complete changelog for 0.7.0 #1443
Complete changelog for 0.7.0 #1443
Changes from 3 commits
49accac
65a25ab
c193e1a
7222703
8de3786
273ce27
85e804b
0da41af
3315714
b77fdf0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Might be good to add this to docs as well? More people would look at those than at the changelogs IMO
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.
Theres not really a clear place to put this in the docs and we already explain all the bootstrappers there :/ I think this is probably the best place for now because the DB node won't return a specific error saying anything is wrong
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.
Sure sounds good. At some point maybe we should introduce
Migration Guides
sections now that it's picking up traction for any breaking changes we introduce. Eg we already have a breaking issue in query so might be good at some point to have a single reference section to describe all migration proceduresThere 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.
Would be good to have it in the docs, then link here rather than saying
described above
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.
same comment as before, should be transparent upgrade for everyone anyways
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.
So very nit, but I think we should lowercase
m3coordinator
,m3query
,m3nsch
since they're names, whereasM3DB
is an abbreviationThere 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, on
# 0.4.8 (2018-10-20)
and before, they just sayDB
,Coordinator
, etc. so we should definitely be consistentThere 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.
To the latter point, I think it's fine that your changelogs become a bit more polished and expressive as time and adoption goes up; probably no reason to be shackled to conventions which were largely ad-hoc decisions