Skip to content

SONARHTML-355 fix(S6853): Move trailing comments to their own lines#566

Merged
zglicz merged 1 commit intomasterfrom
fix/trailing-comments-s139
Feb 4, 2026
Merged

SONARHTML-355 fix(S6853): Move trailing comments to their own lines#566
zglicz merged 1 commit intomasterfrom
fix/trailing-comments-s139

Conversation

@zglicz
Copy link
Contributor

@zglicz zglicz commented Feb 4, 2026

Summary

  • Fixes java:S139 SonarQube issues in LabelHasAssociatedControlCheck
  • Moves trailing comments for Angular/Vue binding attributes to their own lines above each statement
  • Resolves 3 code smell issues on master

Test plan

  • Unit tests pass (mvn test -Dtest=LabelHasAssociatedControlCheckTest)

🤖 Generated with Claude Code

Fixes java:S139 SonarQube issues in LabelHasAssociatedControlCheck.
Trailing comments for Angular/Vue binding attributes are now placed
on the line above each statement.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title fix(S6853): Move trailing comments to their own lines SONARHTML-355 fix(S6853): Move trailing comments to their own lines Feb 4, 2026
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Feb 4, 2026

SONARHTML-355

@sonarqube-next
Copy link

sonarqube-next bot commented Feb 4, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Ruling Report

No changes to ruling expected issues in this PR

@zglicz zglicz requested a review from a team February 4, 2026 13:05
@zglicz zglicz merged commit 3791075 into master Feb 4, 2026
11 checks passed
@zglicz zglicz deleted the fix/trailing-comments-s139 branch February 4, 2026 13:42
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