Skip to content
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

Upgrading version to 0.12.1 #1725

Merged
merged 1 commit into from
Apr 20, 2016
Merged

Conversation

daspecster
Copy link
Contributor

@daspecster daspecster commented Apr 15, 2016

Starting release notes!

$ git log -1 --pretty=%H
355899fb16bbdbc83aea6281e7535e21c70e8319

$ git log 0.12.0..HEAD \
> | grep 'Merge pull request ' \
> | awk -F ' ' '{print $4}' \
> | sort

#1694
#1697
#1698
#1699
#1706
#1708
#1709
#1716
#1717
#1727
#1728
#1729
#1730

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 15, 2016
@daspecster
Copy link
Contributor Author

daspecster commented Apr 15, 2016

Changes not noted in release notes:

Release notes:


Core

Datastore

DNS

Storage

@dhermes
Copy link
Contributor

dhermes commented Apr 16, 2016

@daspecster Thanks for the start, here are some suggested changes.

#1699 isn't really user facing yet, so we can leave it out.


Datastore

DNS

@dhermes
Copy link
Contributor

dhermes commented Apr 16, 2016

@daspecster If you go to https://github.com/GoogleCloudPlatform/gcloud-python/releases/new what kind of error message do you see?

/cc @jgeewax May need your help adding him to a group to resolve.

@daspecster
Copy link
Contributor Author

@dhermes actually that link takes me back to github.com.
Also, I updated the release notes with your doc.

@dhermes
Copy link
Contributor

dhermes commented Apr 16, 2016

OK we can wait on @jgeewax before merging this.

@daspecster
Copy link
Contributor Author

@dhermes Is this good to go now?

@dhermes
Copy link
Contributor

dhermes commented Apr 19, 2016

There have been some merges since Friday so you should rebase against master and re-check the merged PRs. (Thanks for pinging me.)

@daspecster
Copy link
Contributor Author

Ok will do!

@daspecster daspecster reopened this Apr 19, 2016
@daspecster
Copy link
Contributor Author

@dhermes, I updated the release copy. Let me know what you think!

@tseaver
Copy link
Contributor

tseaver commented Apr 19, 2016

@daspecster I tweaked the release notes text a bit.

@daspecster
Copy link
Contributor Author

Awesome! Thanks @tseaver!

@dhermes
Copy link
Contributor

dhermes commented Apr 20, 2016

Go ahead and merge and then create the release.

@daspecster daspecster merged commit 4287c2f into googleapis:master Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants