Hi Raquel, 2015-12-07 14:22 GMT+01:00 Raquel Pau Fernández <[email protected]>:
> Hi Romain, > > First of all, thank you for all your comments and feedback. I really > appreaciate it :). Do you know any other opinions a part from this mailing > list? > > Didnt hear from anyone else :( > About reporting the failures from the Maven plugin. Remember that this tool > is to automatically fix code conventions. Therefore, the problems about > multiple commits to fix them or mailing lists shouldn't appear, right? > > Right but we should take care of the auto-reformatting side - shouldn't impact at all real code. Why? cause we already got noisy-commit cause of reformatting which broke code cause of a single line changed but it was that hidden in the reformatting that we missed it. > I would be interested in offer a conference talk about the benefits of > Walkmod in this project. I would present this formatting issues and a > plugin for CheckStyle. Would it be interesting for the project? > > Think we are still stucked on first point there. > Thank you in advance, > > > 2015-11-30 18:30 GMT+01:00 Romain Manni-Bucau <[email protected]>: > > > 2015-11-30 18:23 GMT+01:00 Raquel Pau Fernández <[email protected]>: > > > > > Hi Romain, > > > > > > There is a maven plugin to execute walkmod during the build process. > > This > > > is a good option to allow people execute walkmod without any > > installation. > > > > > > > > Yep, from some other apache project I know if the build doesnt fail in > such > > a case then it will never be respected - it is quite the case for tomee > > actually where we often commit multiple times to fix just the style which > > is a pain for the dev but patch readers (mailing list) as well. > > > > > > > Moreover, if you run "walkmod check", the tool shows in the command > line > > > the modified files if there is any modification *at syntactic level* > (eg, > > > remove an import). > > > > > > However, the tomee conventions, don't change the AST, just rewrite the > > > files to ensure an specific EOL. Design a checking process at this > level > > > implies to compare the contents of the file before writing with the > > > originals and this could be time consuming. However, if you are > > interested > > > I can offer an "option" to activate in order to apply this comparison. > > > > Let me know if you need I create a new PR with the maven plugin > > > configuration. > > > > > > > > Let's wait other feedbacks before going further but personally it looks > > like a potential nice addition - saw few integrations with > checkstyle/pmd, > > wonder if we can get rid of these plugins using walkmod. > > > > > > > Regards, > > > > > > > > > 2015-11-30 0:27 GMT+01:00 Romain Manni-Bucau <[email protected]>: > > > > > > > Hi Raquel, > > > > > > > > added walkmod.xml, apply is quite interesting and nice. We still need > > to > > > > use win EOL for few files - discovered it thanks to your work!: > > > > > > > > modified: > > > > assembly/openejb-standalone/src/main/resources/service.readme.txt > > > > modified: tomee/apache-tomee/src/main/resources/service.readme.txt > > > > > > > > But other ones uses unix one which is respected by walkmod :). > > > > > > > > > > > > Execution on the whole master makes INFO [main] - Total time: 59.304 > > > > seconds. I would be interested to get a maven integration (mvn > package > > > > would execute it by module and would make the build failing if > > something > > > is > > > > not respected but wouldnt apply anything by default). Is that > something > > > > existing? > > > > > > > > @others: anyone else to test? > > > > > > > > > > > > > > > > Romain Manni-Bucau > > > > @rmannibucau <https://twitter.com/rmannibucau> | Blog > > > > <http://rmannibucau.wordpress.com> | Github < > > > > https://github.com/rmannibucau> | > > > > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber > > > > <http://www.tomitribe.com> > > > > > > > > 2015-11-27 17:21 GMT+01:00 Raquel Pau Fernández <[email protected] > >: > > > > > > > > > Hi, > > > > > > > > > > According the discussion, I have improved walkmod to allow the > > > > > specification of the endline char according a platform. > > > > > > > > > > I have created a PR with the required configuration for walkmod. > You > > > juts > > > > > need to download and execute the 2.1.0 version from walkmod.com. > > > > > > > > > > Here my PR: > > > > > https://github.com/apache/tomee/pull/28 > > > > > > > > > > Have a good weekend. > > > > > > > > > > 2015-11-19 21:36 GMT+01:00 Raquel Pau Fernández < > [email protected] > > >: > > > > > > > > > > > ok! :-) > > > > > > yes, walkmod allows to validate the results locally > > > > > > El 19/11/2015 21:34, "Romain Manni-Bucau" <[email protected] > > > > > > > > escribió: > > > > > > > > > > > >> @Raquel: happy to have a go with walkmod while it matches the > > > > > >> "validate locally" constraint > > > > > >> > > > > > >> Romain Manni-Bucau > > > > > >> @rmannibucau | Blog | Github | LinkedIn | Tomitriber > > > > > >> > > > > > >> > > > > > >> 2015-11-19 12:24 GMT-08:00 Raquel Pau Fernández < > > > [email protected] > > > > >: > > > > > >> > Romain, > > > > > >> > > > > > > >> > Sorry, but I don't know the final conclusions, then :).Do you > > > prefer > > > > > to > > > > > >> try > > > > > >> > other tools first to just format or are you interested in > > adapting > > > > > >> walkmod > > > > > >> > to include your conventions? > > > > > >> > > > > > > >> > In the second case, I would try to find time for it because > for > > my > > > > > >> project, > > > > > >> > this is an opportunity. Finally, just one thing to remind, > > walkmod > > > > > would > > > > > >> > allow not just to format conventions, but also any good > > > programming > > > > > >> > practices that you would to apply (eg. force final variables) > > > > without > > > > > >> > reformatting ;) > > > > > >> > > > > > > >> > Please, let me know the final conclusions :) > > > > > >> > Hi Raquel, > > > > > >> > > > > > > >> > kind of true. Said otherwise: re-formatting is most of the > time > > > > > >> > considered as noise so it should only be applied to the > changed > > > code > > > > > >> > and not the code around or the files not modified at all and > it > > > > should > > > > > >> > be done before the commit IMO. > > > > > >> > > > > > > >> > Starting from scratch it means you can enforce rules easily > with > > > any > > > > > >> > of the tools we spoke about but starting from a big codebase > it > > > > needs > > > > > >> > some more precautions. > > > > > >> > > > > > > >> > > > > > > >> > Romain Manni-Bucau > > > > > >> > @rmannibucau | Blog | Github | LinkedIn | Tomitriber > > > > > >> > > > > > > >> > > > > > > >> > 2015-11-19 10:34 GMT-08:00 Raquel Pau Fernández < > > > > [email protected] > > > > > >: > > > > > >> >> Hi, > > > > > >> >> > > > > > >> >> Walkmod allows to use a formatter, but I think that the > > problems > > > > that > > > > > >> >> people find are as follows: > > > > > >> >> > > > > > >> >> - We don't want a big commit with all the source files > > > reformatted. > > > > > >> >> > > > > > >> >> - We don't want to apply the formatter incrementally to > avoid a > > > set > > > > > of > > > > > >> >> commits just for formatting issues. > > > > > >> >> > > > > > >> >> In fact i have found the same problems in Guava. Google is > not > > > > > >> following > > > > > >> >> its own formatter style. > > > > > >> >> > > > > > >> >> Let me know if I am wrong, > > > > > >> >> > > > > > >> >> Regards > > > > > >> >> El 19/11/2015 18:45, "cchacin" <[email protected]> escribió: > > > > > >> >> > > > > > >> >>> I use the Google Style Guide > > > > > >> >>> <http://google.github.io/styleguide/javaguide.html> (with > > > some > > > > > >> >>> modification like 4 spaces) in this way: > > > > > >> >>> > > > > > >> >>> For Eclipse: > > > > > >> >>> You can configure the formatter with this file: > > > > > >> >>> > > > > > >> >>> > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > https://github.com/google/styleguide/blob/gh-pages/eclipse-java-google-style.xml > > > > > >> >>> > > > > > >> >>> For IntelliJ: > > > > > >> >>> You can configure the formatter with this file > > > > > >> >>> > > > > > >> >>> > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > https://github.com/google/styleguide/blob/gh-pages/intellij-java-google-style.xml > > > > > >> >>> Or use the eclipse formatter plugin > > > > > >> >>> > > > > > >> >>> For Netbeans: use the eclipse formatter plugin > > > > > >> >>> > > > > > >> >>> And there is a maven plugin > > > > > >> >>> > > > > > >> >>> > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > https://maven-java-formatter-plugin.googlecode.com/svn/site/0.4/format-mojo.html > > > > > >> >>> > > > > > >> >>> One suggestion can be to use a pre-commit hook or a webhook > in > > > > your > > > > > >> VCS, > > > > > >> > I > > > > > >> >>> never use walkmod before but probably we can configure a > hook > > > with > > > > > >> > walkmod > > > > > >> >>> also. I think the most important part is to have the common > > > > > >> configuration > > > > > >> >>> file wih the desired styles. > > > > > >> >>> > > > > > >> >>> > > > > > >> >>> > > > > > >> >>> ----- > > > > > >> >>> Carlos Chacin > > > > > >> >>> http://github.com/cchacin > > > > > >> >>> -- > > > > > >> >>> View this message in context: > > > > > >> >>> > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > http://tomee-openejb.979440.n4.nabble.com/PMD-Formatting-tp4676866p4676884.html > > > > > >> >>> Sent from the TomEE Dev mailing list archive at Nabble.com. > > > > > >> >>> > > > > > >> > > > > > > > > > > > > > > > > > > > > > -- > > > > > Raquel Pau > > > > > > > > > > *"Education is the most powerful weapon which you can use to change > > the > > > > > world**"* > > > > > > > > > > Nelson Mandela > > > > > > > > > > personal page: > > > > > http://sites.google.com/site/raquelpau/ > > > > > > > > > > > > > > > > > > > > > -- > > > Raquel Pau > > > > > > *"Education is the most powerful weapon which you can use to change the > > > world**"* > > > > > > Nelson Mandela > > > > > > personal page: > > > http://sites.google.com/site/raquelpau/ > > > > > > > > > -- > Raquel Pau > > *"Education is the most powerful weapon which you can use to change the > world**"* > > Nelson Mandela > > personal page: > http://sites.google.com/site/raquelpau/ >
