Skip to content

JAVASE-189 Upgrade parent pom to version 87.0.0.3057#75

Merged
leonardo-pilastri-sonarsource merged 1 commit intomasterfrom
db/upgrade-parent-pom-to-87.0.0.3057
Apr 2, 2026
Merged

JAVASE-189 Upgrade parent pom to version 87.0.0.3057#75
leonardo-pilastri-sonarsource merged 1 commit intomasterfrom
db/upgrade-parent-pom-to-87.0.0.3057

Conversation

@dorian-burihabwa-sonarsource
Copy link
Copy Markdown
Contributor

No description provided.

@hashicorp-vault-sonar-prod
Copy link
Copy Markdown
Contributor

hashicorp-vault-sonar-prod Bot commented Apr 1, 2026

JAVASE-189

@sonarqube-next
Copy link
Copy Markdown

sonarqube-next Bot commented Apr 1, 2026

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha Bot commented Apr 1, 2026

Summary

Look, I've reviewed a lot of code changes in my mercenary career—espionage ops, corporate sabotage, the usual—but watching myself explain a pom.xml upgrade is deeply weird. (Pause. I mean, I'm literally a fictional character reviewing code for a fictional Java analyzer, so the bar for weird was already pretty low.)

Here's what's happening: the parent POM got upgraded from 86.0.0.3040 to 87.0.0.3057—basically one version bump that pulls in all the parent's dependency updates, plugin configs, and build standards. The 170-file change count? That's not actual logic changes; it's the parent POM machinery auto-updating copyright headers across the codebase (removing year ranges from Copyright (C) 2012-2025Copyright (C), rewording license preambles to match the new parent template). This is the software equivalent of a fresh coat of paint on a building that was already solid. The code itself is untouched.

What reviewers should know

Where to focus: Really just one place—pom.xml itself. That's where the actual version bump lives (line 8). Everything else is mechanical header churn that Maven's parent plugin auto-applied.

What you're looking at: The diff explodes to 2911 lines because license boilerplate got regenerated in ~170 files. You can safely skim this by glancing at one or two file diffs to confirm it's all headers (spoiler: it is), then trust that the rest follow the same pattern. GitHub's diff will let you collapse those files.

Non-obvious bit: The license header rewording is a deliberate change in the parent POM, not a typo fix in this repo. You'll see © SonarSource Sàrl without the year (cleaner) and license text restructured for clarity. If this feels off, check what the new parent version's pom actually specifies.

The risk: Parent POM upgrades silently inherit transitive dependency updates, plugin version bumps, and compiler settings. If this version jump is substantial (87 vs 86 is minor, but check the parent POM's changelog just to be safe), someone should verify that no breaking changes snuck in via dependencies or build tool updates. Run the tests locally to be sure the symbolic execution engine still flies straight.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback


  • 🐟 April Fools' persona mode!

Copy link
Copy Markdown

@sonar-review-alpha sonar-review-alpha Bot left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

This is a pure maintenance PR — one line changed in pom.xml, and the remaining 170 files are mechanical copyright header regeneration driven by the new parent POM. No logic, no behavior, nothing to flag. Safe to merge.

🗣️ Give feedback

@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource merged commit 7a83626 into master Apr 2, 2026
10 checks passed
@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource deleted the db/upgrade-parent-pom-to-87.0.0.3057 branch April 2, 2026 07:50
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