Hi Hervé,
On 10/11/14 9:00 AM, Hervé BOUTEMY wrote:
Log:
- Fixed several Checkstyle errors.
notice that these are errors *reported by* Checkstyle, not really Checkstyle
errors
Checkstyle should add "don't shoot the messenger" disclaimer ;)
I don't shoot the messager....never......already fixed...;-)
r1630934 | khmarbaise | 2014-10-10 18:35:26 +0200 (Fri, 10 Oct 2014) | 2
lines
- Fixed several errors reported by Checkstyle
I report here because next maven-parent version will enable Checkstyle check
on every build: we must all know that we chose the rules that Checkstyle will
report to us. If some rules are too strict, perhaps we'll need to relax them,
but be prepared to avoid blaming Checkstyle :)
I never blame checkstyle ;-...
Notice I prepared some escape ways when checkstyle:check will be run on each
build, to limit frustration:
1. verify phase only, to let us concentrate on making code work before fixing
broken coding conventions
2. @SuppressWarning support (MCHECKSTYLE-241) to ignore individual rules
directly into source code
3. CHECKSTYLE_OFF/ON support (MCHECKSTYLE-247) to ignore rules on complete
code fragments when necessary
4. fine grained global rule or even rule category (MCHECKSTYLE-242) to ignore
rules or rule categories of a whole build with a single property in pom.xml
I hope everybody will enjoy maven-parent 25 like me...
That's what i tried to do and checked against maven-parent 25-SNAPSHOT
....which resulted in those hints...which i have started to clean up...
Kind regards
Karl-Heinz Marbaise
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
