-
Notifications
You must be signed in to change notification settings - Fork 481
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
5.2 release notes #7390
5.2 release notes #7390
Conversation
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.
The content is fine. My review contains mostly minor suggestions and thoughts for the future.
Chances we'll see #7048 in this? (Just sayin' because of those other necessary DB changes...) |
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
It just went to QA so maybe! 😄 But the nice thing about a release train is that there's always another train. 🚂 |
What this PR does / why we need it:
It's the release notes!
Which issue(s) this PR closes:
Closes #7358
Special notes for your reviewer:
I didn't adjust any formatting, but please check the format especially in the installation section at the bottom.
Suggestions on how to test this:
Your favorite text editor!
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
N/A
Is there a release notes update needed for this change?:
N/A
Additional documentation:
N/A