Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add back in deprecated methods in ServiceAccountJwtAccessCredentials #238

Merged
merged 4 commits into from
Mar 27, 2019

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
*/
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
# for free to join this conversation on GitHub. Already have an account? # 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.

3 participants