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

768 commits

This branch
This branch
All branches
Author SHA1 Message Date
Paul Campbell
253526ee97 travis-ci: deploy master only after PR is merged 
Without this, Travis CI will create two deploys, one for the PR and one
after the PR is merged. We only want the merged version to be deployed.

Deploy parent pom.
 2017-05-30 22:39:35 +01:00
Paul Campbell
ebf86a8235 Merge branch 'release/3.1.0' into develop 
* release/3.1.0:
  version: set to 3.2.0-SNAPSHOT
  travis-ci: deploy even when from a pull request
  builder: ExplicitInitilization only applies to object references
  CHANGELOG
  version: set to 3.1.0
 2017-05-30 15:28:15 +01:00
Paul Campbell
b28771989a version: set to 3.2.0-SNAPSHOT 2017-05-30 15:27:29 +01:00
Paul Campbell
90b5f33048 Merge pull request #50 from kemitix/release/3.1.0 
Release 3.1.0
 2017-05-30 15:25:48 +01:00
Paul Campbell
6b2c5f6e77 travis-ci: deploy even when from a pull request 2017-05-30 15:01:28 +01:00
Paul Campbell
4cc70343be builder: ExplicitInitilization only applies to object references 2017-05-30 15:01:05 +01:00
Paul Campbell
96381c9c90 CHANGELOG 2017-05-30 14:42:58 +01:00
Paul Campbell
9e657b36ec version: set to 3.1.0 2017-05-30 14:36:46 +01:00
Paul Campbell
d4a6010474 Merge pull request #27 from kemitix/upgrade-checkstyle 
Upgrade checkstyle to 7.8
 2017-05-30 12:54:44 +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
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