[GitHub] maven-plugins pull request: [MPMD-219] PMD processing errors shoul...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/80 [MPMD-219] PMD processing errors should fail build (skipPmdError=false) This commit adds a "PmdCollectingRenderer" and uses it instead of the report listener, as listeners are unfortunately not informed about errors. If the "skipPmdError" property is true (default), then these processing errors are logged as warnings. You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-219 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/80.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #80 commit d9c298f27c54c7f215fe58a78d87e1519d4dff1c Author: Andreas Dangel <adan...@users.sourceforge.net> Date: 2016-01-25T11:49:33Z [MPMD-219] PMD processing errors should fail build (skipPmdError=false) This commit adds a "PmdCollectingRenderer" and uses it instead of the report listener, as listeners are unfortunately not informed about errors. If the "skipPmdError" property is true (default), then these processing errors are logged as warnings. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-209] Upgrade to PMD 5.3.0
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/50 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-206] Make the source directories...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/47 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-209] Upgrade to PMD 5.3.0
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/50 [MPMD-209] Upgrade to PMD 5.3.0 See http://jira.codehaus.org/browse/MPMD-209 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-209 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/50.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #50 commit 4595bdd24b2d56ee86e82e4347e8dc7ac5fd814a Author: Andreas Dangel adan...@users.sourceforge.net Date: 2015-04-03T08:37:10Z [MPMD-209] Upgrade to PMD 5.3.0 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-206] Make the source directories...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/47 [MPMD-206] Make the source directories configurable Pull request for: https://jira.codehaus.org/browse/MPMD-206 The parameters are manually set to the defaults value instead of using defaultValue option due to https://jira.codehaus.org/browse/MNG-5440 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-206 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/47.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #47 commit eff820b83ed7917ea1b8d6a75a0100f884782e78 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2015-03-04T21:07:30Z [MPMD-206] Make the source directories configurable The parameters are manually set to the defaults value instead of using defaultValue option due to https://jira.codehaus.org/browse/MNG-5440 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-207] Add CPD support for javascr...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/48 [MPMD-207] Add CPD support for javascript and jsp https://jira.codehaus.org/browse/MPMD-207 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-207 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/48.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #48 commit 1d3e75f9e5f571c89ee73b4a2f0fb268e17a2be1 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2015-03-05T19:22:46Z [MPMD-207] Add CPD support for javascript and jsp --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: MPMD-205 Javascript violations won't f...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/46 MPMD-205 Javascript violations won't fail the build See https://jira.codehaus.org/browse/MPMD-205 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-205 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/46.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #46 commit 41723166faefafd6f56a18299212aa92e58b858d Author: Andreas Dangel adan...@users.sourceforge.net Date: 2015-03-02T21:29:30Z [MPMD-205] Add it for pmd-js-check commit 53f9489fdd272a493878a615db507ab804a18f92 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2015-03-04T19:18:16Z [MPMD-205] Fix pmd-js-check commit 7512139dd087edfa5d21a6aa22cb88a88357905b Author: Andreas Dangel adan...@users.sourceforge.net Date: 2015-03-04T19:30:01Z [MPMD-205] Improve the language configuration. Note: Java is now only taken if language is java or null and a targetJdk version is given (which is almost always the case, as targetJdk is by default ${maven.compiler.target}). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-193] CPD's sourceEncoding not se...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/34 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-165] target/site/pmd.html create...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/33 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-171] Rule properties are ignored...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/35 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-187] Report is generated even wh...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/36 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: MPMD-129 Maven PMD plug-in does not ha...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/32 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: MPMD-129 Maven PMD plug-in does not ha...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/32 MPMD-129 Maven PMD plug-in does not have suppressMarker property that is available in PMD itself. https://jira.codehaus.org/browse/MPMD-129 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-129 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/32.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #32 commit 174c51ce33cd83e6093a3e862ca9e296c93f1ded Author: Andreas Dangel adan...@users.sourceforge.net Date: 2014-11-29T15:31:12Z [MPMD-129] Maven PMD plug-in does not have suppressMarker property that is available in PMD itself. Thanks to Svetlomira Manova commit ba475c05231d3f6e31d4d4107773bcd15cbfb86c Author: Andreas Dangel adan...@users.sourceforge.net Date: 2014-11-29T16:10:36Z [MPMD-129] Add a unit test for new suppressMarker property --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-165] target/site/pmd.html create...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/33 [MPMD-165] target/site/pmd.html created with invalid charset=${outputEnc... ...oding} https://jira.codehaus.org/browse/MPMD-165 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-165 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/33.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #33 commit ed09287152358cbe6d0355b7a4ef9b8671894925 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2014-11-29T16:27:57Z [MPMD-165] target/site/pmd.html created with invalid charset=${outputEncoding} --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-193] CPD's sourceEncoding not se...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/34 [MPMD-193] CPD's sourceEncoding not set since PMD 5.1.1 is used That's just a unit test for https://jira.codehaus.org/browse/MPMD-193. After upgrading to PMD 5.2.2 (https://jira.codehaus.org/browse/MPMD-195) this unit test should turn green. You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-193 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/34.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #34 commit 5e975b17cc6418f1d9d8ab4f65eb0cdd8859e091 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2014-11-29T16:49:46Z [MPMD-193] CPD's sourceEncoding not set since PMD 5.1.1 is used - unit test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-171] Rule properties are ignored...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/35 [MPMD-171] Rule properties are ignored when run under MPMD - unit test Just a unit test for https://jira.codehaus.org/browse/MPMD-171 Couldn't reproduce it anymore. The issue was probably fixed with PMD 5.0.3/5.0.4. The bug report is for m-pmd-p 3.0.1, which uses PMD 5.0.2. Related bugs are: https://sourceforge.net/p/pmd/bugs/1074/ (rule priority doesn't work on group definitions) and https://sourceforge.net/p/pmd/bugs/1089/ (When changing priority in a custom ruleset, violations reported twice) You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-171 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/35.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #35 commit 6db8210eb57eb6111ea405adc45b31031712be0d Author: Andreas Dangel adan...@users.sourceforge.net Date: 2014-11-29T17:16:58Z [MPMD-171] Rule properties are ignored when run under MPMD - unit test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-187] Report is generated even wh...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/36 [MPMD-187] Report is generated even when skip is true https://jira.codehaus.org/browse/MPMD-187 This is basically the same as in #21 , but additionally for cpd.skip, too and it adds a integration test. You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins MPMD-187 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/36.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #36 commit f81ac3c1d629888cf91952e6a5c04bf23202d789 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2014-11-29T18:39:14Z [MPMD-187] Report is generated even when skip is true --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
[GitHub] maven-plugins pull request: [MPMD-191] Update to PMD 5.2.1
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/30 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
maven-plugins pull request: [MPMD-168] Add parameter to skip report generat...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/11 - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
maven-plugins pull request: [MPMD-168] Add parameter to skip report generat...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/11 [MPMD-168] Add parameter to skip report generation if there are no violations/duplications. * The report generation for both PMD and CPD is split from the actual execution of these tools. * PMD and CPD are now executed already when canGenerateReport() is called. * The new parameter is called skipEmptyReport and is by default false to remain backwards compatibility. You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins mpmd-168 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/11.patch commit 7cf9f87a1ca14afa6a72e34197781e15e41227f2 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2013-05-04T14:38:36Z [MPMD-168] Add parameter to skip report generation if there are no violations or code duplications. * The report generation for both PMD and CPD is split from the actual execution of these tools. * PMD and CPD are now executed already when canGenerateReport() is called. * The new parameter is called skipEmptyReport and is by default false to remain backwards compatibility. - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
maven-plugins pull request: [MPMD-169] Support multi-threaded mode of PMD 5
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/10 - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
maven-plugins pull request: [MPMD-167] Upgrade pmd version to 5.0.4 to fix ...
Github user adangel closed the pull request at: https://github.com/apache/maven-plugins/pull/9 - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
maven-plugins pull request: [MPMD-167] Upgrade pmd version to 5.0.4 to fix ...
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/9 [MPMD-167] Upgrade pmd version to 5.0.4 to fix CPD performance issues PMD just released 5.0.4 with the needed bugfix. `mvn clean integration-test -Prun-its` is successful for me. Jira ticket: https://jira.codehaus.org/browse/MPMD-167 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins mpmd-167 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/9.patch commit d8ff4060b8565d5883314a517621e4ec3dc98037 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2013-05-01T12:18:22Z [MPMD-167] Upgrade pmd version to 5.0.4 to fix CPD performance issues See also: https://sourceforge.net/p/pmd/bugs/1082/ commit 82dc548853926734ed670358fe29bfb9be5caf7b Author: Andreas Dangel adan...@users.sourceforge.net Date: 2013-05-01T12:20:00Z Remove my duplicated contributor entry - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
maven-plugins pull request: [MPMD-169] Support multi-threaded mode of PMD 5
GitHub user adangel opened a pull request: https://github.com/apache/maven-plugins/pull/10 [MPMD-169] Support multi-threaded mode of PMD 5 PMD 5 supports executing in multi-threaded mode, which should be a performance improvement in case you have multiple cores. This patch removes the workaround introduced while adding PMD 5 support and simply let PMD decided, whether to switch on multi-threaded mode. PMD uses `Runtime.availableProcessors()` to determine the thread-pool size. There is no additional test case added. However, all tests including integration tests work (e.g. `mvn clean integration-test -Prun-its` is successful for me). Jira ticket: https://jira.codehaus.org/browse/MPMD-169 You can merge this pull request into a Git repository by running: $ git pull https://github.com/adangel/maven-plugins mpmd-169 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-plugins/pull/10.patch commit ecbe248019bf5c3fe04fe8c1700861e39936b1d6 Author: Andreas Dangel adan...@users.sourceforge.net Date: 2013-05-01T12:30:38Z [MPMD-169] Support multi-threaded mode of PMD 5 - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org