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

chore: updated JwtClaims.Builder methods to public #396

Merged
merged 1 commit into from Jan 15, 2020

Conversation

rahulKQL
Copy link
Contributor

@rahulKQL rahulKQL commented Jan 15, 2020

Made JwtClaims#Builder methods public so that the user could create JwtClaims instance.

towards #349

Made this public so that the user could create JwtClaims instance.
@rahulKQL rahulKQL requested a review from as a code owner Jan 15, 2020
@googlebot googlebot added the cla: yes label Jan 15, 2020
@codecov
Copy link

@codecov codecov bot commented Jan 15, 2020

Codecov Report

Merging #396 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #396   +/-   ##
=========================================
  Coverage     79.53%   79.53%           
  Complexity      397      397           
=========================================
  Files            27       27           
  Lines          1803     1803           
  Branches        188      188           
=========================================
  Hits           1434     1434           
  Misses          269      269           
  Partials        100      100
Impacted Files Coverage Δ Complexity Δ
...h2_http/java/com/google/auth/oauth2/JwtClaims.java 92.3% <100%> (ø) 8 <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 37f153f...952ad33. Read the comment docs.

@chingor13 chingor13 merged commit 9e5de14 into googleapis:master Jan 15, 2020
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants