Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

Version 2.0.0 #314

Merged
merged 1 commit into from
May 11, 2017
Merged

Version 2.0.0 #314

merged 1 commit into from
May 11, 2017

Conversation

LewisJEllis
Copy link
Contributor

Accepting feedback on the release notes. Will publish first thing tomorrow.

/cc @MaxBittker @benvinegar

@benvinegar
Copy link
Contributor

Based on our previous discussion, this seems right, but I think I'll leave to @MaxBittker to approve.

History.md Outdated
@@ -1,3 +1,16 @@
# 2.0.0 - 5/10/2017
- Properly surface errors to preserve process exit conditions [See #308, #257]
- Node processes with raven will now exit in *all* the same situations as if raven were not present
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe don't emphasize all here when it's not a hard hard guarantee

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just say "the same situations" imo

Copy link
Contributor

@MaxBittker MaxBittker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

considering this is a big version bump, would you consider writing a few of these with annotations about which points someone needs to be aware of for upgrading?

@LewisJEllis
Copy link
Contributor Author

LewisJEllis commented May 11, 2017

which points someone needs to be aware of for upgrading?

Mostly just the one point about shutdown conditions; added a bold callout.

Added the postgres autobreadcrumb removal to this.

@LewisJEllis LewisJEllis merged commit c143117 into master May 11, 2017
@LewisJEllis LewisJEllis deleted the release-2.0.0 branch May 11, 2017 20:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants