Commit graph

568 commits

Author SHA1 Message Date
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
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
b28771989a version: set to 3.2.0-SNAPSHOT 2017-05-30 15:27:29 +01:00
90b5f33048 Merge pull request #50 from kemitix/release/3.1.0
Release 3.1.0
2017-05-30 15:25:48 +01:00
6b2c5f6e77 travis-ci: deploy even when from a pull request 2017-05-30 15:01:28 +01:00
4cc70343be builder: ExplicitInitilization only applies to object references 2017-05-30 15:01:05 +01:00
96381c9c90 CHANGELOG 2017-05-30 14:42:58 +01:00
9e657b36ec version: set to 3.1.0 2017-05-30 14:36:46 +01:00
d4a6010474 Merge pull request #27 from kemitix/upgrade-checkstyle
Upgrade checkstyle to 7.8
2017-05-30 12:54:44 +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
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
061cc5ff91 wercker.yml: added 2017-05-30 00:02:11 +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
a0d58fca81 travis-ci: verbose deploy 2017-05-29 20:31:22 +01:00
aa7f886e6d Merge pull request #48 from kemitix/shippable-config
shippable.yml: added
2017-05-29 15:09:12 +01:00
5b9e32d2eb wercker.yml: rewritten 2017-05-29 14:56:51 +01:00
da5a14c1ce wercker.yml: added 2017-05-29 14:52:04 +01:00
8c5a6e06fd travis-ci: remove defensive checks 2017-05-29 14:46:59 +01:00
02087630c9 shippable.yml: added 2017-05-29 14:40:21 +01:00
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
dbb8bde53e Merge pull request #45 from kemitix/kcr-37-38-explicit-scope
KCR37 & 38 explicit scope
2017-05-29 14:23:32 +01:00
9b571a5a58 travis-ci: only deploy from Travis-CI 2017-05-29 14:20:16 +01:00
1a5ca4ceef KCR37+38: MoveVariableInsideIf: avoid unused private methods 2017-05-29 14:10:04 +01:00
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
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
55a28392f8 KCR39: regressions: MoveVariableInsideIf: method(): use parameter 2017-05-29 13:39:04 +01:00
b6521b809c KCR38: regressions: MoveVariableInsideIf: valid(): use explicit scope 2017-05-29 13:36:30 +01:00
710cfd151b KCR37: regressions: MoveVariableInsideIf: invalid(): use explicit scope 2017-05-29 13:36:11 +01:00
4cca4286a1 KXR36: plugin: DefaultCheckstyleExecutorTest: artifactFile: make local variable 2017-05-29 13:31:56 +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
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
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
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
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
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
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
7229c11b30 KCR29: travis-ci: detect Travis-CI 2017-05-29 12:04:43 +01:00
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
4a4da15d4f KCR29: travis-ci: should only submit coverage report from Travis-CI 2017-05-29 11:45:55 +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