kemitix/kemitix-checkstyle-ruleset
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity
626 commits 1 branch 40 tags 1.3 MiB
Commit graph

234 commits

Author SHA1 Message Date
Paul Campbell
e39dafc62b Upgrade kemitix-maven-tiles to 0.8.1 2018-04-19 07:20:36 +01:00
Paul Campbell
8ef9226a12 Upgrade kemitix-parent to 5.1.0 2018-04-18 20:43:45 +01:00
Paul Campbell
77394ad5d3 builder: refactoring to reduce complexity 2018-03-11 18:37:22 +00:00
Paul Campbell
423255b704 ruleset: update to match kemitix-pmd-ruleset 
FileLength: restrict to 500 lines
JavaNCSS: reduce limits (class & file 250, method 12)
MethodCount: reduce to 10
MethodLength: reduce to 30
 2018-03-11 11:46:17 +00:00
Paul Campbell
4de6b39821 builder: PackageScanner now returns a List rather than a Stream 2018-03-11 09:01:14 +00:00
Paul Campbell
359d15b78f builder: Clean up Configuration 2018-03-11 08:51:03 +00:00
Paul Campbell
d59cb7b879 builder: use FastClasspathScanner for Java 8 too 2018-03-11 08:39:20 +00:00
Paul Campbell
7ba2b557e3 builder: scan class path in java 9 2018-03-10 23:41:35 +00:00
Paul Campbell
5b28ecf762 builder: add dependency on fast-classpath-scanner 2018-03-10 23:39:59 +00:00
Paul Campbell
f333d47613 builder: split Java 8 and 9 class path scanning - java 9 unsupported 2018-03-10 22:52:08 +00:00
Paul Campbell
7ae239809d builder: add dependency on conditional 2018-03-10 22:51:10 +00:00
Paul Campbell
defa19a143 builder: use BuilderConfiguration to provide the classLoader 2018-03-10 08:27:45 +00:00
Paul Campbell
bb13fdaa1c Upgrade checkstyle to 8.7 2018-01-28 19:30:01 +00:00
Paul Campbell
940ba4c2cc Upgrade sevntu-checkstyle to 1.27.0 2018-01-28 19:28:15 +00:00
Paul Campbell
19057140e4 Merge branch 'hotfix/4.0.1' into develop 
* hotfix/4.0.1:
  Remove unused imports
  Add test for checkClasses bean
  Update CHANGELOG
  Add set-version.sh
  Remove use of @PostContruct
  Update README tile version
  Lombok should be provided
  Test should be using interface
  Upgrade kemitix-maven-tiles to 0.3.0
  Add javax.annotation-api for jdk 9 compatibility
  Upgrade lombok to 1.16.20
  version set to 4.0.1
  Add sourceDirectories to avoid scanning generated sources
 2018-01-25 07:22:12 +00:00
Paul Campbell
ede3db53bf Remove unused imports 2018-01-25 06:59:09 +00:00
Paul Campbell
fd63f94a27 Add test for checkClasses bean 2018-01-25 06:55:18 +00:00
Paul Campbell
d3f86fc318 Remove use of @PostContruct 2018-01-24 21:55:52 +00:00
Paul Campbell
5e62e74593 Lombok should be provided 2018-01-24 21:55:14 +00:00
Paul Campbell
3c83b78cea Test should be using interface 2018-01-24 21:47:17 +00:00
Paul Campbell
9983997391 Upgrade kemitix-maven-tiles to 0.3.0 2018-01-24 07:06:56 +00:00
Paul Campbell
1529472940 Add javax.annotation-api for jdk 9 compatibility 2018-01-24 07:04:45 +00:00
Paul Campbell
c5d6af4ffa Upgrade lombok to 1.16.20 2018-01-24 07:04:15 +00:00
Paul Campbell
62c81bd0b3 version set to 4.0.1 2018-01-20 22:27:25 +00:00
Paul Campbell
fb354b9902 Merge branch 'hotfix/4.0.0' into develop 
* hotfix/4.0.0:
  tile use tiles
  ruleset uses UTF-8 encoding
  Regressions use own checkstyle config
  Upgrade maven-checkstyle-plugin to 3.0.0
  Update checkstyle groupId/artifactId/version in support modules
  Update travis-support
  Prevent installation of support modules
  Run builder during package phase
  Add local checkstyle plugin config for builder
  Change ruleset groupId/artifactId to net.kemitix.checkstyle.ruleset
  Simplify groupId and artifactId under net.kemitix.checkstyle
  Restore Header and JavadocPackage checks
  [bug] update the ruleset files and add missing reasons for deactivated rules
  Version set to 3.4.1
 2018-01-17 20:11:12 +00:00
Paul Campbell
c4b79a9831 version set to 4.1.0-SNAPSHOT 2018-01-17 20:07:09 +00:00
Paul Campbell
f9a5684ec5 Prevent installation of support modules 2018-01-17 20:05:53 +00:00
Paul Campbell
9a2297fe6d Run builder during package phase 
This should help to avoid forgetting to update the ruleset and readme in future.
 2018-01-17 20:05:53 +00:00
Paul Campbell
cd4afa6034 Add local checkstyle plugin config for builder 
Builder can't access the current version of the tile during build, so it needs
its own checkstyle config declaration.
 2018-01-17 20:05:53 +00:00
Paul Campbell
68c4ca055a Change ruleset groupId/artifactId to net.kemitix.checkstyle.ruleset 
Add tile maven-tile
Update README
 2018-01-17 20:05:53 +00:00
Paul Campbell
8573a0374a Simplify groupId and artifactId under net.kemitix.checkstyle 2018-01-17 20:04:02 +00:00
Paul Campbell
64f1308004 Restore Header and JavadocPackage checks 2018-01-17 20:00:26 +00:00
Paul Campbell
091e4f139e [bug] update the ruleset files and add missing reasons for deactivated rules 2018-01-17 20:00:26 +00:00
Paul Campbell
7ae51727a2 Upgrade maven-checkstyle-plugin to 3.0.0 2018-01-17 19:10:55 +00:00
Paul Campbell
14a038c9f3 Update checkstyle groupId/artifactId/version in support modules 2018-01-17 18:41:42 +00:00
Paul Campbell
09f1f3eb5f Prevent installation of support modules 2018-01-16 22:36:27 +00:00
Paul Campbell
06b14df6cd Run builder during package phase 
This should help to avoid forgetting to update the ruleset and readme in future.
 2018-01-16 22:28:07 +00:00
Paul Campbell
bbeae5b56c Add local checkstyle plugin config for builder 
Builder can't access the current version of the tile during build, so it needs
its own checkstyle config declaration.
 2018-01-16 22:24:23 +00:00
Paul Campbell
faae9e9afa Change ruleset groupId/artifactId to net.kemitix.checkstyle.ruleset 
Add tile maven-tile
Update README
 2018-01-16 22:23:33 +00:00
Paul Campbell
1770e5aff1 Simplify groupId and artifactId under net.kemitix.checkstyle 2018-01-16 17:38:36 +00:00
Paul Campbell
5b5c0673a0 Restore Header and JavadocPackage checks 2018-01-14 19:13:34 +00:00
Paul Campbell
54fa8f0be4 [bug] update the ruleset files and add missing reasons for deactivated rules 2018-01-14 19:13:13 +00:00
Paul Campbell
5a8cf1b884 Version set to 3.4.1 2018-01-14 18:46:51 +00:00
Paul Campbell
a9d458355f Version set to 3.5.0-SNAPSHOT 2018-01-07 20:20:42 +00:00
Paul Campbell
0a03122f80 Upgrade kemitix-maven-tiles to 0.2.0 2018-01-07 20:06:08 +00:00
Paul Campbell
56d36da8c9 version set to 3.4.0 2018-01-07 20:01:54 +00:00
Paul Campbell
e8452d8c19 Make modules independant of root pom 
* Revert kemitix-maven-tiles to 0.1.1 until release
* Move pom details from root to ruleset pom
 2018-01-07 19:29:36 +00:00
Paul Campbell
309ace6c76 Specify lombok version in parent pom 2018-01-06 19:45:18 +00:00
Paul Campbell
c8c6b1ae57 Disable JavadocPackage check 2018-01-06 08:21:58 +00:00
Paul Campbell
84013ee1ae Disable AvoidStaticImport check 2018-01-06 08:18:39 +00:00
Paul Campbell
e18668dea7 Disable AvoidStarImport check 2018-01-06 08:18:23 +00:00
Paul Campbell
f20517a8e7 Disable Header check 2018-01-06 08:18:05 +00:00
Paul Campbell
fa9cc33a4d Remove requirement for @auther in Type Javadoc 2018-01-06 08:17:38 +00:00
Paul Campbell
87358bfe92 Update builder tests 2018-01-05 22:22:18 +00:00
Paul Campbell
372b8e0f3c Use tiles in builder 2018-01-03 20:35:06 +00:00
Paul Campbell
020635841a Remove kemitix-parent from pom ancestry 2018-01-03 20:27:21 +00:00
Paul Campbell
4328e49c7f version: set to 3.4.0-SNAPSHOT 2017-07-13 15:55:00 +01:00
Paul Campbell
ac7c15694b version: set to 3.3.0 2017-07-13 15:50:06 +01:00
Paul Campbell
90e3652e6e {builder,plugin}:*Test: replace deprecated org.mockito.Matchers.* imports 2017-07-13 14:19:39 +01:00
Paul Campbell
7b144fdb0a AvoidDefaultSerializableInInnerClasses: enable rule 2017-07-13 11:57:44 +01:00
Paul Campbell
3fe3d16d42 FinalizeImplementation: enable rule 2017-07-05 15:30:33 +01:00
Paul Campbell
69ee78e50e WhiteSpaceBeforeArrayInitializer: enable rule 2017-07-04 12:31:37 +01:00
Paul Campbell
55ff66d98b EmptyPublicCtorInClass: enable rule 2017-07-04 12:25:30 +01:00
Paul Campbell
ff2e4b9c95 builder: move dependency management to parent pom 2017-06-17 08:29:05 +01:00
Paul Campbell
ecf22ade2f version: set to 3.3.0-SNAPSHOT 
remove redundant version elements in sub-module poms as they will inherit from the parent pom
 2017-06-12 10:12:48 +01:00
Paul Campbell
f5f69a006f version: set to 3.2.0 2017-06-12 09:48:06 +01:00
Paul Campbell
90901766f5 builder: pom.xml: use project parent pom 2017-06-09 19:43:43 +01:00
Paul Campbell
f5eddc088f builder: pom.xml: upgrade kemitix-parent to 3.0.0 
* Specify Spring Platform as Brussels-SR2
 2017-06-09 19:15:10 +01:00
Paul Campbell
de597e0164 pom.xml: normalise maven pom names 2017-06-02 19:27:14 +01:00
Paul Campbell
2b8533273f Add level 0 Disabled 2017-05-31 12:10:06 +01:00
Paul Campbell
b28771989a version: set to 3.2.0-SNAPSHOT 2017-05-30 15:27:29 +01:00
Paul Campbell
4cc70343be builder: ExplicitInitilization only applies to object references 2017-05-30 15:01:05 +01:00
Paul Campbell
9e657b36ec version: set to 3.1.0 2017-05-30 14:36:46 +01:00
Paul Campbell
f189fb3b84 builder: DefaultPackageScannerTest: make classPath local variable 2017-05-30 12:44:33 +01:00
Paul Campbell
1b318253d2 builder: BuilderConfigurationTest: added 2017-05-30 12:43:34 +01:00
Paul Campbell
0503156bd0 Merge branch 'develop' into upgrade-checkstyle 
* develop:
  wercker.yml: rewritten
  wercker.yml: added
  travis-ci: remove defensive checks
  shippable.yml: added
  travis-ci: only deploy from Travis-CI
  KCR37+38: MoveVariableInsideIf: avoid unused private methods
  KCR39: regressions: MoveVariableInsideIf: method(): use parameter
  KCR38: regressions: MoveVariableInsideIf: valid(): use explicit scope
  KCR37: regressions: MoveVariableInsideIf: invalid(): use explicit scope
  KXR36: plugin: DefaultCheckstyleExecutorTest: artifactFile: make local variable
  builder: ReadmeWriterTest: templateProperties: make local variable
  KCR35: builder: ReadmeWriterTest: fragments: make local variable
  KCR34: builder: ReadmeWriterTest: template: make local variable
  KCR33: builder: ReadmeWriterTest: outputProperties: make local
  KCR31: builder: DefaultRuleReadmeLoaderTest: make field local
  KCr30: CheckstyleWriterTest: checkstyleTemplate: make field local variable
  KCR28: builder: remove empty test class
  builder: CheckstyleWriter: avoid throwing raw exception types
 2017-05-30 12:38:38 +01:00
Paul Campbell
2e8e26c351 builder: refactored and improved test coverage 2017-05-30 12:32:29 +01:00
Paul Campbell
a0ca4958fc builder: use fully qualified classnames in XML files 
In Checkstyle 7.8 a new mechanisim is used to lookup rules by
name. This new method causes an exception to be thrown is there are
more than one classes found that match the rule name.

The solution is to use fully qualified class names in the module tag
in the XML ruleset files.
 2017-05-29 23:55:42 +01:00
Paul Campbell
701b64e2c6 builder: ReadmeWriterTest: templateProperties: make local variable 2017-05-29 13:27:24 +01:00
Paul Campbell
74690d8cdf KCR35: builder: ReadmeWriterTest: fragments: make local variable 2017-05-29 13:25:50 +01:00
Paul Campbell
d754fdc0a3 KCR34: builder: ReadmeWriterTest: template: make local variable 2017-05-29 13:23:37 +01:00
Paul Campbell
c4f302d9e5 KCR33: builder: ReadmeWriterTest: outputProperties: make local 
Fixes #33
 2017-05-29 13:22:57 +01:00
Paul Campbell
b04d131245 Merge pull request #43 from kemitix/kcr-31-builder-defaultrulereadmeloadertest-fragment-localvariable 
KCR31: builder: DefaultRuleReadmeLoaderTest: make field local
 2017-05-29 13:16:35 +01:00
Paul Campbell
27806fb2a2 KCR31: builder: DefaultRuleReadmeLoaderTest: make field local 
* fragment (Fixes  #31)
* templateProperties (Fixes #32)
 2017-05-29 12:27:40 +01:00
Paul Campbell
f7867283ff KCr30: CheckstyleWriterTest: checkstyleTemplate: make field local variable 2017-05-29 12:24:51 +01:00
Paul Campbell
82999a87fc Merge branch 'develop' into kcr-28-avoid-throwing-raw-exception-types 
* develop:
  KCR29: travis-ci: detect Travis-CI
  KCR29: travis-ci: dump $TRAVIS and other env
  KCR29: travis-ci: should only submit coverage report from Travis-CI
  KCR29: builder: Rule: sortByName(): use explicit scoping
 2017-05-29 12:11:08 +01:00
Paul Campbell
9d2895269d KCR29: builder: Rule: sortByName(): use explicit scoping 2017-05-29 11:31:13 +01:00
Paul Campbell
9ac59206a0 KCR28: builder: remove empty test class 2017-05-29 11:26:24 +01:00
Paul Campbell
81768ff9f9 builder: CheckstyleWriter: avoid throwing raw exception types 
Instead of throwing RuntimeException or IOException, throw
CheckstyleWriterException or TemplateNotFoundException.
 2017-05-29 11:23:48 +01:00
Paul Campbell
4a0ee1de46 ruleset: MoveVariableInsideIf: enabled for 4-tweaks and above 2017-05-28 19:56:43 +01:00
Paul Campbell
6072613b44 pom.xml: upgrade sevntu to 1.24.0 
* Fixes bug in ForbidWildcardAsReturnType which is now enabled again
* Additional pluginRepositories config is no longer required
 2017-05-28 19:52:24 +01:00
Paul Campbell
df9926c428 Merge branch 'master' into develop 
* master:
  travis--ci: update key to match passphrase
  travis--ci: batch import signing key
  travis-ci: replace signing key to match passphrase
  .travis.yml: CI_DEPLOY_GPG_PASSPHRASE (4/4)
  .travis.yml: CI_DEPLOY_GPG_KEYNAME (3/4)
  .travis.yml: CI_DEPLOY_PASSWORD (2/4)
  .travis.yml: replace encrypted data (1/4)
  travis-ci: update location of travis-settings.xml
  travis-ci: use a shell-safe password
  travis-ci: make shell scripts executable (again)
  travis-ci: remove comments
  travis-ci: updated and add encrypted gpg credentials
  pom.xml: version set to 2.1.2
 2017-05-28 18:06:02 +01:00
Paul Campbell
2706045dfb travis-ci: updated and add encrypted gpg credentials 2017-05-28 14:54:23 +01:00
Paul Campbell
c4be689fcc version: set to 3.0.1 2017-05-27 20:45:11 +01:00
Paul Campbell
2584ed4bb3 version: set to 3.1.0-SNAPSHOT 2017-05-27 16:48:15 +01:00
Paul Campbell
5af558544e pom.xml: version set to 3.0.0 2017-05-27 16:45:22 +01:00
Paul Campbell
c14c9ea8ca KCR22: builder: sort rules alphabetically 2017-05-27 14:04:19 +01:00
Paul Campbell
1b27a84bd6 version: set to 3.0.0-SNAPSHOT 2017-05-27 08:58:11 +01:00
Paul Campbell
d1d174d5a7 API Change: plugin: level is specified as a configuration parameter 
* merge all level-based goals into a single 'check' goal
* specify the level to apply as a configuration parameter
* add remaining unit tests for plugin
 2017-05-27 08:53:56 +01:00
Paul Campbell
6f9f07709f README.md,application.yml: fix typo 
ArrayTrailingCommand and FinalLocalVariable had typo in level (TWEAK
when it should have been TWEAKS)
 2017-05-26 09:17:29 +01:00