Skip to content

SONARHTML-445 Remove S5257 from Sonar Way - #796

Merged
erwan-leforestier-sonarsource merged 1 commit into
masterfrom
chore/sonarhtml-445-remove-s5257-from-sonar-way
Aug 24, 2026
Merged

SONARHTML-445 Remove S5257 from Sonar Way#796
erwan-leforestier-sonarsource merged 1 commit into
masterfrom
chore/sonarhtml-445-remove-s5257-from-sonar-way

Conversation

@erwan-leforestier-sonarsource

@erwan-leforestier-sonarsource erwan-leforestier-sonarsource commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes S5257 from the HTML Sonar Way profile while retaining the rule for custom quality profiles.
RSPEC PR: https://github.com/SonarSource/rspec/pull/7933

Changes

  • Remove S5257 from the Sonar Way profile definition
  • Verify Sonar Way does not activate S5257
  • Verify S5257 remains registered in the HTML rule repository

@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource requested a review from a team August 20, 2026 13:12
@hashicorp-vault-sonar-prod

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

Copy link
Copy Markdown

SONARHTML-445

@github-actions

Copy link
Copy Markdown
Contributor

Ruling Report

No changes to ruling expected issues in this PR

@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource marked this pull request as ready for review August 20, 2026 13:27
@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource force-pushed the chore/sonarhtml-445-remove-s5257-from-sonar-way branch from abe78fd to 2339a6c Compare August 20, 2026 13:28
@gitar-bot

gitar-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Removes rule S5257 from the Sonar Way quality profile while keeping it available for custom profiles. 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

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource merged commit 90297be into master Aug 24, 2026
13 checks passed
@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource deleted the chore/sonarhtml-445-remove-s5257-from-sonar-way branch August 24, 2026 09:16
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