Repository navigation
Conversation
- Declare jsr305 as compileOnly in r2dbc and the R2DBC providers (MySQL, MariaDB, PostgreSQL, MSSQL, ClickHouse). They use javax.annotation.Nullable, which Guava >= 33.4 no longer brings in. - Ignore Pulsar major updates: Pulsar 5 requires Java 17. - Drop the localstack 0.12.8 test image, which was removed from Docker Hub; retag 0.11.3 for the custom-tag legacy test instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015e4ZbWmbHNUD21D5bnXdVd
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
|
No actionable comments were generated in the recent review. Walkthrough
Merge Risk:
|
The grouped Dependabot PR #12158 fails about 20 module checks. This PR fixes the causes on our side, so that the group can be recreated and pass.
Changes
javax.annotation.Nullableno longer resolves after the Guava bumpGuava 33.4+ no longer depends on
com.google.code.findbugs:jsr305.r2dbcand the R2DBC providers in MySQL, MariaDB, PostgreSQL, MSSQL and ClickHouse importjavax.annotation.Nullable. Until now they compiled only because core's Guava brought jsr305 in transitively. With Guava 33.8.0,:testcontainers-r2dbc:compileJavafails, and so does every module that depends on it.→ This PR declares
jsr305:3.0.2ascompileOnlyin those six modules. The published POMs don't change.Pulsar 5 requires Java 17
pulsar-client5.0.1 is published for Java 17+, and the module is still built against Java 8, so dependency resolution fails.→ This PR adds a Dependabot rule to ignore major
org.apache.pulsar:*updates.localstack/localstack:0.12.8was removed from Docker HubEvery 0.12.x tag now returns
manifest unknown, soLegacyModeTestfails on every full run. It passed on 2026-10-27 and failed on #12161 on 2026-11-13.→ This PR removes the 0.12 test image. The "0.12" non-legacy case duplicated the 0.11 case, which stays. The custom-tag legacy test now retags
0.11.3.Verification
I couldn't verify this locally because Maven Central returned 429 (Too Many Requests) while resolving dependencies, so CI on this PR is the check. After this merges, ask Dependabot to recreate #12158 (its
recreatecommand), so the group is rebuilt on top of this fix.Generated by Claude Code
Summary by CodeRabbit