Skip to content

SONARJAVA-6713 Update rule S2696: Unwrap parenthesis in static field - #5881

Merged
guillemsarda merged 1 commit into
masterfrom
fix/SONARJAVA-6713-cover-fn
Aug 5, 2026
Merged

SONARJAVA-6713 Update rule S2696: Unwrap parenthesis in static field#5881
guillemsarda merged 1 commit into
masterfrom
fix/SONARJAVA-6713-cover-fn

Conversation

@guillemsarda

@guillemsarda guillemsarda commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary by Gitar

  • Rule updates:
    • Updated StaticFieldUpateCheck to unwrap parenthesis when checking static field modifications.
    • Added test cases in StaticFieldUpateCheckSample.java covering parenthesized static field assignments and increments.

This will update automatically on new commits.

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

SONARJAVA-6713

@gitar-bot

gitar-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Updates StaticFieldUpateCheck to unwrap parenthesis when checking static field modifications, accompanied by new test cases covering parenthesized assignments and increments. No issues found.

Options

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

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 Aug 4, 2026

Copy link
Copy Markdown

@guillemsarda
guillemsarda merged commit f386392 into master Aug 5, 2026
19 checks passed
@guillemsarda
guillemsarda deleted the fix/SONARJAVA-6713-cover-fn branch August 5, 2026 08:23
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