Skip to content

SONARJAVA-7222 Fix S9434 and S9435 issues - #6386

Merged
aurelien-coet-sonarsource merged 1 commit into
masterfrom
ac/fix-qg
Oct 22, 2026
Merged

aurelien-coet-sonarsource merged 1 commit into
masterfrom
ac/fix-qg

Conversation

@aurelien-coet-sonarsource

Copy link
Copy Markdown
Contributor

No description provided.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Fix S9434 and S9435 issues SONARJAVA-7222 Fix S9434 and S9435 issues Oct 22, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

SONARJAVA-7222

@gitar-bot

gitar-bot Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Code Review ✅ Approved

🟢 Low risk · Localized ternary and this-qualification rewrites preserve the same branches and target, so they have little effect on runtime behavior or interfaces.

Refactors code to address SonarQube findings S9434 and S9435 under SONARJAVA-7222. No issues found.

Verification Rules 2/2 met PRs 1 inspected

Functional validation No results

Rules 2 passed 1 not applicable

✅ Internal artifact registries
✅ Repository license compliance

Broader impact PRs 1 inspected
Cross-PR 1 PR inspected 0 open impacts

PR 6385: [DO NOT MERGE] ci-autofix: Investigate red master CI 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

Copy link
Copy Markdown
Contributor

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

LGTM!

@aurelien-coet-sonarsource
aurelien-coet-sonarsource merged commit a8f1874 into master Oct 22, 2026
20 checks passed
@aurelien-coet-sonarsource
aurelien-coet-sonarsource deleted the ac/fix-qg branch October 22, 2026 08:03
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