Skip to content

JS-2127 Demote S6440 from Sonar way#7611

Open
erwan-leforestier-sonarsource wants to merge 1 commit into
masterfrom
chore/js-2127-demote-s6440-from-sonar-way
Open

JS-2127 Demote S6440 from Sonar way#7611
erwan-leforestier-sonarsource wants to merge 1 commit into
masterfrom
chore/js-2127-demote-s6440-from-sonar-way

Conversation

@erwan-leforestier-sonarsource

@erwan-leforestier-sonarsource erwan-leforestier-sonarsource commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

This removes S6440 from the built-in Sonar way profile while keeping the rule available outside that default profile.

RSPEC PR: https://github.com/SonarSource/rspec/pull/7537

Changes

  • refresh S6440 metadata so it is no longer part of Sonar way
  • regenerate the Sonar way aggregate profile without S6440
  • add targeted tests for Sonar way activation and generated S6440 recommendation metadata

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Jul 21, 2026

Copy link
Copy Markdown

JS-2127

@datadog-sonarsource

This comment has been minimized.

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Ruling Report

No changes to ruling expected issues in this PR

@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource force-pushed the chore/js-2127-demote-s6440-from-sonar-way branch from c7a9c81 to 50f7f08 Compare July 21, 2026 15:15
@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource force-pushed the chore/js-2127-demote-s6440-from-sonar-way branch from 50f7f08 to e3a8c7f Compare July 21, 2026 15:35
@gitar-bot

gitar-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Removes S6440 from the default Sonar way profile by updating rule metadata and regenerating the profile. 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

@erwan-leforestier-sonarsource
erwan-leforestier-sonarsource marked this pull request as ready for review July 21, 2026 15:37
@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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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