[mina-sshd] branch master updated: GH-356: Simplify mvn commands in Github workflow

[twolf]

This is an automated email from the ASF dual-hosted git repository.

twolf pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mina-sshd.git


The following commit(s) were added to refs/heads/master by this push:
     new baf853fe9 GH-356: Simplify mvn commands in Github workflow
baf853fe9 is described below

commit baf853fe976e714af0b178e9f970a2baa7f973c3
Author: Thomas Wolf <tw...@apache.org>
AuthorDate: Sat May 27 14:30:13 2023 +0200

    GH-356: Simplify mvn commands in Github workflow
    
    Somehow the continuation lines did't work. No idea why.
---
 .github/workflows/master-build.yml | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/master-build.yml 
b/.github/workflows/master-build.yml
index 604b1a949..d0fb7fb3e 100644
--- a/.github/workflows/master-build.yml
+++ b/.github/workflows/master-build.yml
@@ -76,9 +76,7 @@ jobs:
 
       - name: Build and test with maven
         # Skip all static checks, they were already done in the compile jobs
-        run: |
-          mvn -B --errors --activate-profiles ci --no-transfer-progress 
package \
-            -Dcheckstyle.skip -Dpmd.skip -Dcpd.skip -Dfindbugs.skip 
-Dspotbugs.skip
+        run: mvn -B --errors --activate-profiles ci --no-transfer-progress 
package
 
       - name: Archive test results and logs
         # if: success() || failure() to also get the test results on 
successful runs.
@@ -152,7 +150,6 @@ jobs:
       - name: Build and deploy with maven
         # Must be quoted, ! is special in yaml
         if: "! env.SKIP_DEPLOYMENT"
-        # Skip tests and all static checks -- this was done already in 
previous jobs
         # NEXUS_USERNAME and NEXUS_PASSWORD are used in ~/.m2/settings.xml 
created by
         # the setup-java action. The two secrets are organization-wide secrets 
that
         # were enabled by Apache Infra for our repository.
@@ -162,7 +159,4 @@ jobs:
         # Our root POM overwrites the release repository with an invalid value 
to prevent
         # accidental release deployments as an additional safety measure. The 
snapshot
         # repo from the Apache parent POM is not overridden.
-        run: |
-          mvn -B --errors --activate-profiles ci --no-transfer-progress clean 
deploy \
-            -DdeployAtEnd \
-            -DskipTests -Dcheckstyle.skip -Dpmd.skip -Dcpd.skip 
-Dfindbugs.skip -Dspotbugs.skip
+        run: mvn -B --errors --activate-profiles ci --no-transfer-progress 
deploy -DskipTests -DdeployAtEnd