DataGenerator icon indicating copy to clipboard operation
DataGenerator copied to clipboard

What PMD modules to use?

Open yukaReal opened this issue 10 years ago • 6 comments

Now we are declare to use:

  • Java -> Basic
  • Java -> Strict Exceptions

Actually we compliant with:

  • Java -> Basic

There are more other modules:

  • java ** Android ** Basic ** Braces ** Clone Implementation ** Code Size ** Comments ** Controversial ** Coupling ** Design ** Empty Code ** Finalizer ** Import Statements ** J2EE ** JavaBeans ** JUnit ** Jakarta Commons Logging ** Java Logging ** Migration ** Naming ** Optimization ** Strict Exceptions ** String and StringBuffer ** Security Code Guidelines ** Type Resolution ** Unnecessary ** Unused Code

So, what modules we are going to use?

yukaReal avatar Aug 06 '14 05:08 yukaReal

Currently, only "braces" and "exceptions" are enabled.

bryantrobbins avatar Aug 14 '14 22:08 bryantrobbins

Other than android and controversial, we can go through them tomorrow and pick few targeting 2.1 .

Thoughts ? On Aug 14, 2014 6:35 PM, "Bryan Robbins" [email protected] wrote:

Currently, only "braces" and "exceptions" are enabled.

— Reply to this email directly or view it on GitHub https://github.com/FINRAOS/DataGenerator/issues/91#issuecomment-52254176 .

mibrahim avatar Aug 14 '14 23:08 mibrahim

Btw... some of those may overlap with checkstyle . On Aug 14, 2014 7:50 PM, "Mohamed Ibrahim" [email protected] wrote:

Other than android and controversial, we can go through them tomorrow and pick few targeting 2.1 .

Thoughts ? On Aug 14, 2014 6:35 PM, "Bryan Robbins" [email protected] wrote:

Currently, only "braces" and "exceptions" are enabled.

— Reply to this email directly or view it on GitHub https://github.com/FINRAOS/DataGenerator/issues/91#issuecomment-52254176 .

mibrahim avatar Aug 14 '14 23:08 mibrahim

  1. Talk to @mpeter28 about the rule that he likes to disable.
  2. Propose some good new rules to add
  3. Review with the group and update

mibrahim avatar Feb 06 '15 14:02 mibrahim

@SMxJrz

mibrahim avatar Feb 06 '15 14:02 mibrahim

Untagging this from 2.2 unless there is more interest.

bryantrobbins avatar Oct 23 '15 13:10 bryantrobbins