NET-668 Modify S7130: Add vbnet rspec #4499
Conversation
|
|
|
Converted to DRAFT so we don't merge before the rule is merged. |
| }, | ||
| "tags": [ | ||
| ], | ||
| "defaultSeverity": "Major", |
There was a problem hiding this comment.
@pavel-mikula-sonarsource re-seeing this - is it really major?
Maybe medium?
There was a problem hiding this comment.
Severities are "Info","Minor","Major","Critical","Blocker". Major is maped to Medium,
https://docs.sonarsource.com/sonarqube/9.9/user-guide/issues/
It's closer to "unused parameter" and "why the hell is this here" than "this is just too long line and that makes it hard to read"
Review
A dedicated reviewer checked the rule description successfully for: