Hi Michael, >> Next question is how we would "clean up" the code base? >> My suggestion would be to do one single "clean up run and git push" without >> further code changes. >> What do you think?
Sounds great! Maybe you can tell us the timeframe when you are planning to do the cleanup to avoid any merge conflicts? Best regards, Jens. -----Original Message----- From: Bolz, Michael [mailto:[email protected]] Sent: Dienstag, 13. August 2013 08:37 To: [email protected] Subject: Re: code style Hi, Weekend is over and the result for line wrapping looks as follows: +6: "line wrapping policy" +1: "80 chars" +3: "120 chars" +2: "135 chars" So the winner is "line wrapping=true" with "120 chars". I adapted the checkstyle configuration and the usage in the parent pom (see commit "2849d4eba978b83155c6ac92cc46087134ff1ffe"). Currently checkstyle has to be run manually (e.g. "mvn checkstyle:check") or over the quality build profile ("mvn package -Pbuild.quality"). Next question is how we would "clean up" the code base? My suggestion would be to do one single "clean up run and git push" without further code changes. What do you think? Kind regards and a nice week Michael
