-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Cutting release 0.22.1 of datastore. #2911
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
Conversation
This release is per @bjwatson request (via email). There have only been 2 relevant commits since the last release (and a 3rd commit in this PR):
They are in PRs #2870 and #2876. The change in #2870 is just part of removing Then change in #2876 is also (mostly) opaque to users. It swaps our hand generated |
@daspecster @jonparrott @tseaver Due to #2870, we may need a release of core. The only
but they have some definite implications on all dependent packages (i.e. this release of Given that, it may make sense to create a release branch for WDYT all? I made such a branch in my repo via:
/cc @bjwatson |
If we can finish the @dhermes, I can take a stab at that over the weekend if you can tell me roughly everything that needs to be done. (My life is boring and I don't have much for NYE plans. haha) If it's kind of an urgent matter then cherry picking the commits is probably the only way to go. |
@daspecster The @bjwatson asked that this release go out in 2016. |
Ok, I wasn't sure just how much was left. When I looked at it before, it didn't seem like there was much. I guess cherry picking is the path then! |
+1 for cherry-picking only the relevant changes w/o a |
OK. @tseaver and @daspecster WDYT of me just running:
This PR should still be merged but the tag won't be created from PS I am not at all a fan of the "cherry-pick release branch", but it's small scope here and is a "business need" for some reason I don't totally recall right now. If it starts to become a habit we should push back and rethink the process. |
SGTM, and I agree with keeping an eye out for this in the future. |
This still needs to be merged to make sure |
Cutting release 0.22.1 of datastore.
No description provided.