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
c14c9ea8ca
KCR22: builder: sort rules alphabetically
2017-05-27 14:04:19 +01:00
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
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
