Skip to content

Conversation

@tcoffee-google
Copy link
Contributor

Addresses #1594.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 14, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 81.162% when pulling 42afcd1 on tcoffee-google:translate_api_key into 7c20c6f on GoogleCloudPlatform:master.

@tcoffee-google
Copy link
Contributor Author

@gguuss PTAL

Copy link
Contributor

@gguuss gguuss left a comment

Choose a reason for hiding this comment

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

LGTM.

@gguuss
Copy link
Contributor

gguuss commented Jun 22, 2017

Code looked good but Travis test is failing. Once it passes the change should resolve the issue.

@tcoffee-google
Copy link
Contributor Author

Travis is currently hitting a build time limit on OS X.

@tcoffee-google tcoffee-google merged commit a898acc into googleapis:master Jun 23, 2017
@tcoffee-google tcoffee-google deleted the translate_api_key branch June 23, 2017 00:34
tcoffee-google added a commit to tcoffee-google/google-cloud-java that referenced this pull request Jun 30, 2017
tcoffee-google added a commit that referenced this pull request Jul 14, 2017
* Make Translate auth documentation clear and consistent

Additional updates prompted by #1594 / #2147.

* Fix doc link

* Fix Spanner doc example

* Work around Javadoc bug

https://bugs.openjdk.java.net/browse/JDK-8130754
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