Skip to content

chore: declare jspecify in jar parent and ignore if unused#13704

Merged
lqiu96 merged 2 commits into
mainfrom
fix-jspecify-dependency-analysis
Jul 8, 2026
Merged

chore: declare jspecify in jar parent and ignore if unused#13704
lqiu96 merged 2 commits into
mainfrom
fix-jspecify-dependency-analysis

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jul 8, 2026

Copy link
Copy Markdown
Member

Declare org.jspecify:jspecify as a dependency in google-cloud-jar-parent so that all client libraries inherit it, resolving "used undeclared dependency" issues caused by the new JSpecify changes.

Also add org.jspecify:jspecify to ignoredUnusedDeclaredDependencies in java-shared-config to prevent "unused declared dependency" warnings/failures in modules that do not use it yet (e.g. gRPC modules)

Declare org.jspecify:jspecify as a dependency in google-cloud-jar-parent
so that all client libraries inherit it, resolving "used undeclared dependency"
issues caused by the new gapic-generator-java JSpecify onboarding.

Also add org.jspecify:jspecify to ignoredUnusedDeclaredDependencies in
java-shared-config to prevent "unused declared dependency" warnings/failures
in modules that do not use it yet.

TAG=agy
CONV=d6bb9d61-0aa4-4d9f-af2b-ce10da21c3bd
@lqiu96 lqiu96 changed the title deps(parents): declare jspecify in jar parent and ignore if unused chore: declare jspecify in jar parent and ignore if unused Jul 8, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds the org.jspecify:jspecify dependency to the parent POM (google-cloud-jar-parent/pom.xml) and configures it as an ignored unused declared dependency in the shared configuration POM (java-shared-config/java-shared-config/pom.xml). There are no review comments, and I have no feedback to provide.

Update parent POMs that override ignoredUnusedDeclaredDependencies to
explicitly include org.jspecify:jspecify. This prevents "unused declared dependency"
failures in submodules (like grpc-* modules) that inherit JSpecify from
google-cloud-jar-parent but do not use it, when their parent POM overrides
the global ignore list defined in java-shared-config.

TAG=agy
CONV=d6bb9d61-0aa4-4d9f-af2b-ce10da21c3bd
@lqiu96
lqiu96 requested a review from nnicolee July 8, 2026 20:19
@lqiu96
lqiu96 marked this pull request as ready for review July 8, 2026 20:19
@lqiu96
lqiu96 requested review from a team as code owners July 8, 2026 20:19
@lqiu96
lqiu96 merged commit eb5648d into main Jul 8, 2026
328 of 338 checks passed
@lqiu96
lqiu96 deleted the fix-jspecify-dependency-analysis branch July 8, 2026 20:27
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.

2 participants