Skip to content

Conversation

@kolea2
Copy link
Contributor

@kolea2 kolea2 commented Mar 27, 2019

To be deleted at a later time.

@kolea2 kolea2 requested a review from a team as a code owner March 27, 2019 19:03
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 27, 2019
@codecov
Copy link

codecov bot commented Mar 27, 2019

Codecov Report

Merging #238 into master will decrease coverage by 0.1%.
The diff coverage is 0%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #238 +/- ## ============================================ - Coverage 78.33% 78.23% -0.11%  Complexity 330 330 ============================================ Files 21 21 Lines 1445 1447 +2 Branches 158 158 ============================================ Hits 1132 1132 - Misses 235 237 +2  Partials 78 78
Impacted Files Coverage Δ Complexity Δ
...uth/oauth2/ServiceAccountJwtAccessCredentials.java 74.63% <0%> (-1.1%) 36 <0> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 49b39ae...3d5346b. Read the comment docs.

@kolea2 kolea2 requested a review from sduskis March 27, 2019 19:09
* @deprecated Use {@link #newBuilder()} instead. This constructor will either be deleted or made
* private in a later version.
*/
private ServiceAccountJwtAccessCredentials(String clientId, String clientEmail,
Copy link
Contributor

Choose a reason for hiding this comment

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

IMHO, this constructor should continue to be private.

@kolea2 kolea2 merged commit 88b3810 into googleapis:master Mar 27, 2019
@kolea2 kolea2 deleted the jwt-fix branch March 27, 2019 19:35
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.

3 participants