search

jmecosta / sonar-fsharp-plugin

F# SonarQube (TM) plugin - support for F#
GNU Lesser General Public License v3.0
52 stars 14 forks source link

Number of recommended rules off #58

Open milbrandt opened 5 years ago

milbrandt commented 5 years ago

The recommended way is called Sonar way. Is this naming ok? By default 38 code smells are enabled, the only one bug rule is disabled.

For customized profiles, the number of disabled rules compared to the recommended Sonar way is displayed. For this plugin it is not taken into account that the bug rule is already disabled in the recommended "Sonar way rules". For 21 rules anabled the expected difference to 38 default enabled rules is 17, not 18.

SonarWayNotIncluded-1 0 6 110

I just tested with typescript plugin by creating a copy and disabling some rules. There the behavior is as expected (would be 17).

jmecosta commented 5 years ago

its probably because there is a template rule, that is likely not counted

jmecosta commented 5 years ago

regarding the severity/type this work hasnt been done. setting those to major, then users can do what they like. Sonar Way is the default naming for the readonly profile. tought thats SonarQube team naming, it doesnt mean we need follow it strictly. but since its there its thre