Skip to content

SONARRUBY-96 Update rule metadata#78

Merged
dorian-burihabwa-sonarsource merged 1 commit intomasterfrom
tt/update-rule-metadata
Nov 27, 2025
Merged

SONARRUBY-96 Update rule metadata#78
dorian-burihabwa-sonarsource merged 1 commit intomasterfrom
tt/update-rule-metadata

Conversation

@tomasz-tylenda-sonarsource
Copy link
Contributor

No description provided.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Update rule metadata SONARRUBY-96 Update rule metadata Nov 26, 2025
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Nov 26, 2025

SONARRUBY-96

@sonarqube-next
Copy link

Quality Gate passed Quality Gate passed for 'Ruby'

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

{
"name": "Sonar way",
"ruleKeys": [
"ParsingError",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was removed and I do not know why, so I am planning to keep it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was removed intentionally: https://github.com/SonarSource/rspec/pull/5935

Choose a reason for hiding this comment

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

That is actually a problem and we received no explanation as to why it was removed. Let me go an check

Copy link
Contributor

Choose a reason for hiding this comment

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

After discussion, let's exclude S2260 from SonarWay for Ruby

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource merged commit 33fe8f2 into master Nov 27, 2025
12 checks passed
@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource deleted the tt/update-rule-metadata branch November 27, 2025 16:04
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