[GitHub] commons-lang issue #285: make checkstyle config more portable (no maven coup...

2017-09-09 Thread osiegmar
Github user osiegmar commented on the issue: https://github.com/apache/commons-lang/pull/285 > Could you update the pull request with your original change duplicated in that portion, please? Done. Thanks for pointing that out! ---

[GitHub] commons-lang issue #285: make checkstyle config more portable (no maven coup...

2017-09-08 Thread osiegmar
Github user osiegmar commented on the issue: https://github.com/apache/commons-lang/pull/285 I haven't checked the configuration of the maven site plugin in detail. Does it really generates a site for the current branch? I doubt that, as I've seen subversion commands in the config

[GitHub] commons-lang issue #285: make checkstyle config more portable (no maven coup...

2017-09-08 Thread osiegmar
Github user osiegmar commented on the issue: https://github.com/apache/commons-lang/pull/285 Removing the SuppressionFilter results in thousands of errors. Maybe you have a stale checkstyle cache file `target/cachefile`? ---

[GitHub] commons-lang pull request #285: make checkstyle config more portable (no mav...

2017-09-08 Thread osiegmar
GitHub user osiegmar opened a pull request: https://github.com/apache/commons-lang/pull/285 make checkstyle config more portable (no maven coupling) You can merge this pull request into a Git repository by running: $ git pull https://github.com/osiegmar/commons-lang portable

[GitHub] commons-lang pull request #284: remove redundant modifiers (support recent c...

2017-09-08 Thread osiegmar
GitHub user osiegmar opened a pull request: https://github.com/apache/commons-lang/pull/284 remove redundant modifiers (support recent checkstyle versions) You can merge this pull request into a Git repository by running: $ git pull https://github.com/osiegmar/commons-lang