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

Make Translate auth documentation clear and consistent #2208

Merged
merged 5 commits into from
Jul 14, 2017

Conversation

tcoffee-google
Copy link
Contributor

Additional updates prompted by #1594 / #2147.

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

Coverage Status

Coverage increased (+0.004%) to 81.139% when pulling b89d7d1 on tcoffee-google:clarify_adc_auth into 2a92207 on GoogleCloudPlatform:master.

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.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling c37d55e on tcoffee-google:clarify_adc_auth into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 320a61b on tcoffee-google:clarify_adc_auth into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 8353ebd on tcoffee-google:clarify_adc_auth into ** on GoogleCloudPlatform:master**.

@tcoffee-google tcoffee-google merged commit 8e074f3 into googleapis:master Jul 14, 2017
@tcoffee-google tcoffee-google deleted the clarify_adc_auth branch July 14, 2017 05:26
vkedia added a commit that referenced this pull request Apr 14, 2018
This method already has a snippet. It was manually changed to put under <code> block instead of @code in #2208 due to javadoc bug
https://bugs.openjdk.java.net/browse/JDK-8130754

When I reran the add snippets tool to add the snippets for a different method, it did not detect the existing snippet on readWriteTransaction and again added the same snippet under @code block which fails. My change removes the duplicate snippet but next time we add a new spanner snippet, we will again run into this issue. Ideally we should fix the add snippet tool to handle this case.
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