Skip to content

feat(auth): add JSpecify Null annotations to Auth#13842

Merged
nnicolee merged 10 commits into
mainfrom
feat/jspecify-auth
Jul 23, 2026
Merged

feat(auth): add JSpecify Null annotations to Auth#13842
nnicolee merged 10 commits into
mainfrom
feat/jspecify-auth

Conversation

@nnicolee

@nnicolee nnicolee commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR migrates the Google Auth Library for Java (google-auth-library-java) to use JSpecify 1.0.0 nullability annotations (@NullMarked and @Nullable), replacing the previous javax.annotation references.

Key Changes

JSpecify 1.0.0 Annotation Adoption in Auth:

  • NullMarked Semantics: Applied @NullMarked at the class and package levels across core auth classes (in credentials, oauth2_http, appengine, and cab-token-generator modules) to define non-null by default semantics.
  • Annotation Replacement: Replaced imports of javax.annotation.Nullable with org.jspecify.annotations.Nullable.
  • Dependency Management: Updated Maven pom.xml files and Bazel BUILD.bazel configurations to declare and manage the org.jspecify:jspecify dependency.

Verification Results

gemini-code-assist[bot]

This comment was marked as off-topic.

@sonarqubecloud

This comment was marked as outdated.

@sonarqubecloud

This comment was marked as outdated.

@nnicolee
nnicolee force-pushed the feat/jspecify-auth branch from 76c939a to afdc8c6 Compare July 21, 2026 04:39
@nnicolee
nnicolee force-pushed the feat/jspecify-auth branch from afdc8c6 to 432c5d0 Compare July 21, 2026 04:43
@nnicolee
nnicolee requested a review from lqiu96 July 21, 2026 15:12
Comment thread google-auth-library-java/pom.xml Outdated
@lqiu96

lqiu96 commented Jul 22, 2026

Copy link
Copy Markdown
Member

/gemini review

gemini-code-assist[bot]

This comment was marked as resolved.

@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 22, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 22, 2026
@nnicolee

Copy link
Copy Markdown
Contributor Author

/gemini review

gemini-code-assist[bot]

This comment was marked as resolved.

@nnicolee
nnicolee marked this pull request as ready for review July 23, 2026 04:14
@nnicolee
nnicolee requested review from a team as code owners July 23, 2026 04:14
@nnicolee
nnicolee merged commit f6f24d4 into main Jul 23, 2026
320 of 322 checks passed
@nnicolee
nnicolee deleted the feat/jspecify-auth branch July 23, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants