Skip to content

SONARJAVA-7303 Update Sonar JDT - #6489

Merged
aurelien-coet-sonarsource merged 1 commit into
masterfrom
ac/update-jdt
Nov 6, 2026
Merged

aurelien-coet-sonarsource merged 1 commit into
masterfrom
ac/update-jdt

Conversation

@aurelien-coet-sonarsource

@aurelien-coet-sonarsource aurelien-coet-sonarsource commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary by Gitar

  • Dependency update:
    • Bumped jdt-package from 1.9.0.1840 to 1.10.0.12296 in java-frontend/pom.xml

This will update automatically on new commits.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Update Sonar JDT SONARJAVA-7303 Update Sonar JDT Nov 5, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

SONARJAVA-7303

@aurelien-coet-sonarsource

Copy link
Copy Markdown
Contributor Author

/pvf

@hashicorp-vault-sonar-prod

Copy link
Copy Markdown
Contributor

📊 Performance Validation complete
🆕 2 new · 🗑️ 0 lost issues
View dashboard

@aurelien-coet-sonarsource

Copy link
Copy Markdown
Contributor Author

/pvf

@gitar-bot

gitar-bot Bot commented Nov 6, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Updating the JDT package could alter Java parsing or analysis behavior, potentially changing reported issues or compatibility across projects.

Bumps the Sonar JDT dependency jdt-package from 1.9.0.1840 to 1.10.0.12296 in java-frontend/pom.xml. No issues found.

Verification Rules 1/2 met Repos 1 inspected · PRs 2 inspected
Rules 1 failed 1 passed 1 not applicable

❌ Repository license compliance - LICENSE.txt contains SONAR Source-Available License v1.0 text, but the central exception for sonar-java requires LGPL-3.0; a maintainer should confirm which license applies, as I can't choose legal text.
✅ Internal artifact registries

Functional validation No results

Broader impact Repos 1 inspected PRs 2 inspected
Cross-repo 1 repository inspected 0 open impacts

SonarSource/sonar-java-symbolic-execution Inspected

Cross-PR 2 PRs inspected 0 open impacts

PR 6459: SONARJAVA-7279 Resolve the JDK API of the target release through the new jdt-package option Inspected
PR 169: SONARJAVA-7278 Override ASTParser to expose the JDK API of a requested release (JEP 247) Inspected

Auto-approval Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next

sonarqube-next Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

@rombirli rombirli 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.

As discussed the 2 newly introduced issues seem correct, LGTM

@aurelien-coet-sonarsource
aurelien-coet-sonarsource merged commit 17e62e6 into master Nov 6, 2026
20 checks passed
@aurelien-coet-sonarsource
aurelien-coet-sonarsource deleted the ac/update-jdt branch November 6, 2026 14:55
@hashicorp-vault-sonar-prod

Copy link
Copy Markdown
Contributor

📊 Performance Validation complete
🆕 2 new · 🗑️ 0 lost issues
View dashboard

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