Re: Linter / Code Formatting
I like the idea of CheckStyle because it has an Eclipse Plugin. I can also add an Eclipse Formatter XML to the project somewhere. The default Eclipse Formatter does not comply with the Oracle Style Guide. On Mon, Feb 19, 2018 at 11:47 AM, Emmanuel Lécharny wrote: > > > Le 19/02/2018 à 16:47, Jonathan Valliere a écrit : > > Any interest in adding support for a linter to automatically check for > > formatting / quality issues and formatting the code to enforce consistent > > style across the entire codebase? > > > > I would add twi things : > > - months ago, I ran the code against Java 8 with the javadoc linter. I > spent days fixing all the missing Javadoc and incorrect ones. The > javadoc linter is on by default now on. It's possible to disbale it, > there is a commented property in the pom.xml file : > > ... > > > ... > > - Checkstyle is pretty good at verifying the code format. We don't > currently use it in MINA, but would we want to set it up, I'll suggest > we mimic what we did in teh Apache Directory project, where it's on by > default. It's based on a checkstyle configuration, and a plugin : > > https://gitbox.apache.org/repos/asf?p=directory-buildtools.git;a=tree;f= > checkstyle-configuration;h=67f80df6c18be403fcecd13464b685 > 4bd31eb89e;hb=HEAD > > https://gitbox.apache.org/repos/asf?p=directory-ldap- > api.git;a=blob;f=pom.xml;h=3bfaec9c08bba75ff71b8810bbf092 > d101280d2e;hb=HEAD > > with : > > > org.apache.maven.plugins > maven-checkstyle-plugin > > true > false > > > > validate > validate > > check > > > > > > > That works pretty well. > > -- > Emmanuel Lecharny > > Symas.com > directory.apache.org > >
Re: Linter / Code Formatting
Le 19/02/2018 à 16:47, Jonathan Valliere a écrit : > Any interest in adding support for a linter to automatically check for > formatting / quality issues and formatting the code to enforce consistent > style across the entire codebase? > I would add twi things : - months ago, I ran the code against Java 8 with the javadoc linter. I spent days fixing all the missing Javadoc and incorrect ones. The javadoc linter is on by default now on. It's possible to disbale it, there is a commented property in the pom.xml file : ... ... - Checkstyle is pretty good at verifying the code format. We don't currently use it in MINA, but would we want to set it up, I'll suggest we mimic what we did in teh Apache Directory project, where it's on by default. It's based on a checkstyle configuration, and a plugin : https://gitbox.apache.org/repos/asf?p=directory-buildtools.git;a=tree;f=checkstyle-configuration;h=67f80df6c18be403fcecd13464b6854bd31eb89e;hb=HEAD https://gitbox.apache.org/repos/asf?p=directory-ldap-api.git;a=blob;f=pom.xml;h=3bfaec9c08bba75ff71b8810bbf092d101280d2e;hb=HEAD with : org.apache.maven.plugins maven-checkstyle-plugin true false validate validate check That works pretty well. -- Emmanuel Lecharny Symas.com directory.apache.org
Re: Linter / Code Formatting
Le 19/02/2018 à 16:47, Jonathan Valliere a écrit : > Any interest in adding support for a linter to automatically check for > formatting / quality issues and formatting the code to enforce consistent > style across the entire codebase? We can use Checkstyle for that purpose. And, yes, that would be a good idea. For the record, we are following teh standard java convention for the formatting. -- Emmanuel Lecharny Symas.com directory.apache.org