Checkstyle : Whitespace rules replacement in squid?

[Laurent Tourreau]

Hi guys

In Checkstyle rules coverage page : http://dist.sonarsource.com/reports/coverage/checkstyle.html

I see whitespace related rules are considered as "rejected":

• NoWhitespaceAfter
  
• NoWhitespaceBefore
  
• WhitespaceAround
  

However there are implemented in Checkstyle plugin :

• checkstyle:com.puppycrawl.tools.checkstyle.checks.whitespace.NoWhitespaceAfterCheck
  
• checkstyle:com.puppycrawl.tools.checkstyle.checks.whitespace.NoWhitespaceBeforeCheck
  
• checkstyle:com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

I have two questions:

1/ By considered them as "rejected" does it mean you deliberately don't want to implement a squid replacement?

2/ If a squid replacement will be possible, when do you think it will be implemented on Java Plugin?

Thanks for your answers.

Regards

Laurent TOURREAU

[Michael Gumowski]

Hello Laurent,

That's exactly the meaning of 'rejected'. These rules won't be implemented by any squid replacement. So, no replacement planned.

We implemented some formatting related rules in the past, but we now try to focus our effort on bug-related rules, providing more value for end users than formatting issues.

Cheers,

Michael GUMOWSKI | SonarSource

Software Developer @ Language Team
http://sonarsource.com

--
You received this message because you are subscribed to the Google Groups "SonarQube" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sonarqube+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/sonarqube/13f1a2fe-d38f-4f1f-a951-b4aa00c0f357%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Laurent Tourreau]

Thanks Michael for this clear explanation.

This also means we won't be able to follow those rules in SonarLint (contrary to the previous SonarQube plugin), since it checks only squid rules.

Regards

Laurent TOURREAU