-
-
Notifications
You must be signed in to change notification settings - Fork 364
Update 2.3 Release Notes #409
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
Update 2.3 Release Notes #409
Conversation
I think this is ready for review and merge if someone has a chance to take a look. cc @zarr-developers/core-devs |
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.
I just read through these release notes and found no obvious problems or omissions.
Thanks Joe. I feel like the 80-20 rule applies here though. Writing the
prototype was the easy part.
…On Thu, Mar 21, 2019 at 6:48 PM Joe Hamman ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In docs/release.rst
<#409 (comment)>:
> @@ -36,7 +38,8 @@ Enhancements
* Adds the MongoDB-backed :class:`zarr.storage.MongoDBStore` class enabling a
MongoDB database to be used as the backing store for an array or group.
- By :user:`Joe Hamman <jhamman>`, :issue:`299`, :issue:`372`.
+ By :user:`Joe Hamman <jhamman>`, :user:`John Kirkham <jakirkham>`,
+ :issue:`299`, :issue:`372`, :issue:`401`.
can we add @nbren12 <https://github.com/nbren12> to the list of
contributors here? He prototyped this and though his work doesn't show up
in the commit record, he deserves 95% of the credit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#409 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABUokuOeg1MX_HtSgiR4ZUUY_Fw6nu1Eks5vZDZdgaJpZM4bGqgb>
.
|
Have responded to the reviews. Please let me know if there is anything else needed 🙂 |
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
Thanks all. Will go ahead and cut the release now. |
Adds some content that was left out of the 2.3 release notes that has been included since the 2.2.0 release.
TODO:
tox -e docs
)