Skip to content

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