Commit graph

110 commits

Author SHA1 Message Date
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
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
2706045dfb travis-ci: updated and add encrypted gpg credentials 2017-05-28 14:54:23 +01:00
c4be689fcc version: set to 3.0.1 2017-05-27 20:45:11 +01:00
2584ed4bb3 version: set to 3.1.0-SNAPSHOT 2017-05-27 16:48:15 +01:00
5af558544e pom.xml: version set to 3.0.0 2017-05-27 16:45:22 +01:00
c14c9ea8ca KCR22: builder: sort rules alphabetically 2017-05-27 14:04:19 +01:00
1b27a84bd6 version: set to 3.0.0-SNAPSHOT 2017-05-27 08:58:11 +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
7c774a5057 {builder,plugin}: pom.xml: upgrades
* kemitix-parent: 2.4.0 -> 2.8.0
* spring-boot: 1.5.1 -> 1.5.3
* lombok: 1.16.12 -> 1.16.16 (reverting to kemitix-parent version)
* assertj: 3.6.2 -> 3.8.0
* maven: 3.3.9 -> 3.5.0
* mojo-executor: 2.2.0 -> 2.3.0
2017-05-25 16:28:35 +01:00
db496848cc pom.xml: version set to 2.4.0-SNAPSHOT 2017-04-02 16:16:50 +01:00
2004fc60d6 pom.xml: version set to 2.3.0 2017-04-02 16:14:23 +01:00
42985446a0 ForbidWildcardAsReturnType: disabled
When used with org.immutables:value, this check will kill the
Checkstyle process by throwing a NullPointerException.
2017-04-02 14:17:57 +01:00
8b1627df0b LICENSE.txt: updated 2017-04-02 13:45:59 +01:00
efa9b98378 pom.xml: version set to 2.3.0-SNAPSHOT 2017-03-11 17:44:34 +00:00
cc5d8f7ed1 pom.xml: version set to 2.2.0 2017-03-11 17:42:31 +00:00
f9ee73f148 RegexpOnFilename: Add a new rule to detect if there are any sync conflict files
Fails if there there are any Syncthing or Dropbox sync conflicts on any *.java file.
2017-03-11 16:55:34 +00:00
0e2f762cd0 builder: pom.xml: remove dependency on plugin
The checkstyle plugin wouldn't have been installed yet.
2017-02-26 17:42:08 +00:00
eebe90b44b builder: pom.xml: rename map-builder property for consistency 2017-02-26 17:41:24 +00:00
7134a476a2 builder: pom.xml: kemitix-spring-parent replaced with kemitix-parent:2.4.0 2017-02-26 17:40:34 +00:00
4de68a84ce builder:CheckstyleWriterTest: use system's file separator 2017-02-08 19:50:54 +00:00
df4c951142 builder:CheckstyleWriter: use system's line separator 2017-02-08 19:40:53 +00:00
829cc5dff6 pom.xml: version set to 2.2.0-SNAPSHOT 2017-02-08 12:30:18 +00:00
961eb2c1db pom.xml: version set to 2.1.0 2017-02-08 12:26:55 +00:00
5fc556aaf2 builder:pom.xml: upgrade dependencies
* spring-boot 1.5.1
* assertj 3.6.2
* mapstream 2.3.5
2017-02-08 11:29:48 +00:00
7e2a0790ff pom.xml: version set to 2.1.0-SNAPSHOT 2017-01-14 17:53:26 +00:00
c5d0bda1ca pom.xml: version set to 2.0.0 2017-01-14 17:47:14 +00:00
67557955cc plugin: provide goals for each ruleset level 2017-01-14 17:04:18 +00:00
dc639da079 plugin: added 2017-01-13 22:53:01 +00:00
c254afcca7 builder:pom.xml: clean up 2017-01-13 22:53:01 +00:00
f39f1a88db builder: configure coveralls coverage reporting 2017-01-10 20:34:23 +00:00
e5c22f47e0 builder:DefaultRuleReadmeLoaderTest: added 2017-01-10 20:29:33 +00:00
2b08488e38 builder:ReadmeFragmentNotFoundException: added 2017-01-10 20:29:04 +00:00