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

359 commits

This branch
This branch
All branches
Author SHA1 Message Date
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
d870720ed6 Merge pull request #49 from kemitix/full-package-name-in-xml 
Use full class name in xml
 2017-05-30 06:35:30 +01:00
Paul Campbell
061cc5ff91 wercker.yml: added 2017-05-30 00:02:11 +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
a0d58fca81 travis-ci: verbose deploy 2017-05-29 20:31:22 +01:00
Paul Campbell
aa7f886e6d Merge pull request #48 from kemitix/shippable-config 
shippable.yml: added
 2017-05-29 15:09:12 +01:00
Paul Campbell
5b9e32d2eb wercker.yml: rewritten 2017-05-29 14:56:51 +01:00
Paul Campbell
da5a14c1ce wercker.yml: added 2017-05-29 14:52:04 +01:00
Paul Campbell
8c5a6e06fd travis-ci: remove defensive checks 2017-05-29 14:46:59 +01:00
Paul Campbell
02087630c9 shippable.yml: added 2017-05-29 14:40:21 +01:00
Paul Campbell
cb1ed7c63b Merge pull request #47 from kemitix/only-deploy-from-travisci 
travis-ci: only deploy from Travis-CI
 2017-05-29 14:33:17 +01:00
Paul Campbell
dbb8bde53e Merge pull request #45 from kemitix/kcr-37-38-explicit-scope 
KCR37 & 38 explicit scope
 2017-05-29 14:23:32 +01:00
Paul Campbell
9b571a5a58 travis-ci: only deploy from Travis-CI 2017-05-29 14:20:16 +01:00
Paul Campbell
1a5ca4ceef KCR37+38: MoveVariableInsideIf: avoid unused private methods 2017-05-29 14:10:04 +01:00
Paul Campbell
b205e01a72 Merge pull request #46 from kemitix/kcr-39-unused-parameter 
KCR39: regressions: MoveVariableInsideIf: method(): use parameter
 2017-05-29 14:08:13 +01:00
Paul Campbell
94a5faebcb Merge pull request #44 from kemitix/kcr-33-34-35-36-local-variables 
KCR33,34,35 & 36 local variables
 2017-05-29 14:07:53 +01:00
Paul Campbell
55a28392f8 KCR39: regressions: MoveVariableInsideIf: method(): use parameter 2017-05-29 13:39:04 +01:00
Paul Campbell
b6521b809c KCR38: regressions: MoveVariableInsideIf: valid(): use explicit scope 2017-05-29 13:36:30 +01:00
Paul Campbell
710cfd151b KCR37: regressions: MoveVariableInsideIf: invalid(): use explicit scope 2017-05-29 13:36:11 +01:00
Paul Campbell
4cca4286a1 KXR36: plugin: DefaultCheckstyleExecutorTest: artifactFile: make local variable 2017-05-29 13:31:56 +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
aa8c5f2c84 Merge pull request #42 from kemitix/kcr-30-builder-checkstylewriter-checkstyletemplate-localvariable 
KCR30: CheckstyleWriterTest: checkstyleTemplate: make field local variable
 2017-05-29 13:16:22 +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
d7454a1155 Merge pull request #40 from kemitix/kcr-28-avoid-throwing-raw-exception-types 
KCR 28 avoid throwing raw exception types
 2017-05-29 12:22:10 +01:00
Paul Campbell
68b2f0456c Merge branch 'develop' into upgrade-checkstyle 
* 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:29 +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
2f2ea3afeb Merge pull request #41 from kemitix/kcr-29-builder-rule-explicit-scope 
KCR29: builder: Rule: sortByName(): use explicit scoping
 2017-05-29 12:09:18 +01:00
Paul Campbell
7229c11b30 KCR29: travis-ci: detect Travis-CI 2017-05-29 12:04:43 +01:00
Paul Campbell
dd17826a60 KCR29: travis-ci: dump $TRAVIS and other env 
Trying to detect when running in Travis CI and not Shippable so only
Travis-CI submits to coveralls.
 2017-05-29 11:51:29 +01:00
Paul Campbell
4a4da15d4f KCR29: travis-ci: should only submit coverage report from Travis-CI 2017-05-29 11:45:55 +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
da5461555e pom.xml: upgrade checkstyle to 7.8 2017-05-29 09:34:18 +01:00
Paul Campbell
e634dba450 regressions: ExplicitInitialization: added 
Regression added in preparation for checkstyle adding an option in
7.8.x to allow primatives to be initialised to their default values.
 2017-05-29 09:19:13 +01:00
Paul Campbell
17d82f5885 sample-parent: set level to 5-complexity 
plugin-sample: overrides level back down to 2-naming
 2017-05-29 09:16:17 +01:00
Paul Campbell
5876d723e3 Merge pull request #26 from kemitix/upgrade-dependencies 
Upgrade dependencies
 2017-05-28 20:10:07 +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
403d4b6504 pom.xml: upgrade checkstyle to 7.7 2017-05-28 18:18:05 +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
adfece4b6e travis--ci: update key to match passphrase 2017-05-28 17:56:17 +01:00