Commit graph

120 commits

Author SHA1 Message Date
058df393d0 Remove regressions module - not adding anything valuable 2018-05-14 20:30:13 +01:00
0b0f11430e Upgrade tiles-maven-plugin to 2.11 2018-04-19 21:04:55 +01:00
ea2c909a32 EmptyLineSeparator: disable 2018-04-19 21:04:55 +01:00
77394ad5d3 builder: refactoring to reduce complexity 2018-03-11 18:37:22 +00:00
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
4de6b39821 builder: PackageScanner now returns a List rather than a Stream 2018-03-11 09:01:14 +00:00
359d15b78f builder: Clean up Configuration 2018-03-11 08:51:03 +00:00
d59cb7b879 builder: use FastClasspathScanner for Java 8 too 2018-03-11 08:39:20 +00:00
7ba2b557e3 builder: scan class path in java 9 2018-03-10 23:41:35 +00:00
f333d47613 builder: split Java 8 and 9 class path scanning - java 9 unsupported 2018-03-10 22:52:08 +00:00
defa19a143 builder: use BuilderConfiguration to provide the classLoader 2018-03-10 08:27:45 +00:00
ede3db53bf Remove unused imports 2018-01-25 06:59:09 +00:00
fd63f94a27 Add test for checkClasses bean 2018-01-25 06:55:18 +00:00
d3f86fc318 Remove use of @PostContruct 2018-01-24 21:55:52 +00:00
3c83b78cea Test should be using interface 2018-01-24 21:47:17 +00:00
62c81bd0b3 version set to 4.0.1 2018-01-20 22:27:25 +00:00
faae9e9afa Change ruleset groupId/artifactId to net.kemitix.checkstyle.ruleset
Add tile maven-tile
Update README
2018-01-16 22:23:33 +00:00
5b5c0673a0 Restore Header and JavadocPackage checks 2018-01-14 19:13:34 +00:00
54fa8f0be4 [bug] update the ruleset files and add missing reasons for deactivated rules 2018-01-14 19:13:13 +00:00
c8c6b1ae57 Disable JavadocPackage check 2018-01-06 08:21:58 +00:00
84013ee1ae Disable AvoidStaticImport check 2018-01-06 08:18:39 +00:00
e18668dea7 Disable AvoidStarImport check 2018-01-06 08:18:23 +00:00
f20517a8e7 Disable Header check 2018-01-06 08:18:05 +00:00
fa9cc33a4d Remove requirement for @auther in Type Javadoc 2018-01-06 08:17:38 +00:00
87358bfe92 Update builder tests 2018-01-05 22:22:18 +00:00
90e3652e6e {builder,plugin}:*Test: replace deprecated org.mockito.Matchers.* imports 2017-07-13 14:19:39 +01:00
7b144fdb0a AvoidDefaultSerializableInInnerClasses: enable rule 2017-07-13 11:57:44 +01:00
3fe3d16d42 FinalizeImplementation: enable rule 2017-07-05 15:30:33 +01:00
69ee78e50e WhiteSpaceBeforeArrayInitializer: enable rule 2017-07-04 12:31:37 +01:00
55ff66d98b EmptyPublicCtorInClass: enable rule 2017-07-04 12:25:30 +01:00
2b8533273f Add level 0 Disabled 2017-05-31 12:10:06 +01:00
4cc70343be builder: ExplicitInitilization only applies to object references 2017-05-30 15:01:05 +01:00
f189fb3b84 builder: DefaultPackageScannerTest: make classPath local variable 2017-05-30 12:44:33 +01:00
1b318253d2 builder: BuilderConfigurationTest: added 2017-05-30 12:43:34 +01:00
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
2e8e26c351 builder: refactored and improved test coverage 2017-05-30 12:32:29 +01:00
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
701b64e2c6 builder: ReadmeWriterTest: templateProperties: make local variable 2017-05-29 13:27:24 +01:00
74690d8cdf KCR35: builder: ReadmeWriterTest: fragments: make local variable 2017-05-29 13:25:50 +01:00
d754fdc0a3 KCR34: builder: ReadmeWriterTest: template: make local variable 2017-05-29 13:23:37 +01:00
c4f302d9e5 KCR33: builder: ReadmeWriterTest: outputProperties: make local
Fixes #33
2017-05-29 13:22:57 +01:00
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
27806fb2a2 KCR31: builder: DefaultRuleReadmeLoaderTest: make field local
* fragment (Fixes  #31)
* templateProperties (Fixes #32)
2017-05-29 12:27:40 +01:00
f7867283ff KCr30: CheckstyleWriterTest: checkstyleTemplate: make field local variable 2017-05-29 12:24:51 +01:00
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
9d2895269d KCR29: builder: Rule: sortByName(): use explicit scoping 2017-05-29 11:31:13 +01:00
9ac59206a0 KCR28: builder: remove empty test class 2017-05-29 11:26:24 +01:00
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
4a0ee1de46 ruleset: MoveVariableInsideIf: enabled for 4-tweaks and above 2017-05-28 19:56:43 +01:00
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