You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ac...@apache.org on 2019/11/06 18:22:20 UTC
[camel-quarkus] branch master updated: chore: remove checkstyle
leftovers
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git
The following commit(s) were added to refs/heads/master by this push:
new 24d4e8e chore: remove checkstyle leftovers
new 2680982 Merge pull request #380 from lburgazzoli/checkstyle-cleanup
24d4e8e is described below
commit 24d4e8e5e54ce094c3f29f2ded43e9af689c2118
Author: lburgazzoli <lb...@gmail.com>
AuthorDate: Wed Nov 6 17:10:43 2019 +0100
chore: remove checkstyle leftovers
---
.github/workflows/pr-validate.yml | 4 ++--
docs/modules/ROOT/pages/contributor-guide.adoc | 3 +--
poms/bom-deployment/pom.xml | 2 --
poms/build-parent/pom.xml | 2 --
4 files changed, 3 insertions(+), 8 deletions(-)
diff --git a/.github/workflows/pr-validate.yml b/.github/workflows/pr-validate.yml
index 362bd50..f96e040 100644
--- a/.github/workflows/pr-validate.yml
+++ b/.github/workflows/pr-validate.yml
@@ -13,5 +13,5 @@ jobs:
uses: actions/setup-java@v1
with:
java-version: 1.8
- - name: mvn -Psourcecheck,license
- run: ./mvnw -fae -B -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn -Dcheckstyle.failOnViolation=true validate -Psourcecheck,license
+ - name: mvn validate
+ run: ./mvnw -fae -B -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn clean validate -Plicense
diff --git a/docs/modules/ROOT/pages/contributor-guide.adoc b/docs/modules/ROOT/pages/contributor-guide.adoc
index 1f553f8..6af60db 100644
--- a/docs/modules/ROOT/pages/contributor-guide.adoc
+++ b/docs/modules/ROOT/pages/contributor-guide.adoc
@@ -115,7 +115,6 @@ may consider omitting explicit `nameBase` in some cases.
from the root of the source tree to add your extension to the autogenerated list of extensions.
11. Before sending a pull request, please squash your commits and run
- `mvn validate -Plicense,sourcecheck -Dcheckstyle.failOnViolation=true` to add license headers to the new files and
- to make sure that your code is properly formatted.
+ `mvn validate -Psourcecheck` to add license headers to the new files.
Good luck!
diff --git a/poms/bom-deployment/pom.xml b/poms/bom-deployment/pom.xml
index 811e154..8dce796 100644
--- a/poms/bom-deployment/pom.xml
+++ b/poms/bom-deployment/pom.xml
@@ -35,8 +35,6 @@
<properties>
<camel-quarkus.version>0.3.2-SNAPSHOT</camel-quarkus.version><!-- kept in sync with project.version by the release plugin -->
- <maven-checkstyle-plugin.version>3.0.0</maven-checkstyle-plugin.version>
- <maven-checkstyle.version>7.6.1</maven-checkstyle.version>
<jandex-maven-plugin.version>1.0.6</jandex-maven-plugin.version>
<maven-resources-plugin.version>3.1.0</maven-resources-plugin.version>
<docker-maven-plugin.version>0.3.0</docker-maven-plugin.version>
diff --git a/poms/build-parent/pom.xml b/poms/build-parent/pom.xml
index 230a8c5..aa06d98 100644
--- a/poms/build-parent/pom.xml
+++ b/poms/build-parent/pom.xml
@@ -39,8 +39,6 @@
<!-- maven-surefire-plugin -->
<failIfNoTests>false</failIfNoTests>
- <maven-checkstyle-plugin.version>3.0.0</maven-checkstyle-plugin.version>
- <maven-checkstyle.version>7.6.1</maven-checkstyle.version>
<jandex-maven-plugin.version>1.0.6</jandex-maven-plugin.version>
<maven-resources-plugin.version>3.1.0</maven-resources-plugin.version>
<docker-maven-plugin.version>0.3.0</docker-maven-plugin.version>