[ 
https://issues.apache.org/jira/browse/MWRAPPER-88?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17656270#comment-17656270
 ] 

ASF GitHub Bot commented on MWRAPPER-88:
----------------------------------------

bmarwell commented on code in PR #79:
URL: https://github.com/apache/maven-wrapper/pull/79#discussion_r1065060233


##########
.github/workflows/shellcheck-posix.yml:
##########
@@ -27,7 +27,7 @@ jobs:
     runs-on: ubuntu-latest
     env:
       SHELLCHECK_VERSION: '0.9.0'
-      SHELLCHECK_OPTS: '-e SC2009'
+      SHELLCHECK_OPTS: '-e SC1091 -e SC2009'

Review Comment:
   Done.





> mvnw is not POSIX compatible
> ----------------------------
>
>                 Key: MWRAPPER-88
>                 URL: https://issues.apache.org/jira/browse/MWRAPPER-88
>             Project: Maven Wrapper
>          Issue Type: Bug
>          Components: Maven Wrapper Scripts
>    Affects Versions: 3.1.1
>            Reporter: Benjamin Marwell
>            Assignee: Benjamin Marwell
>            Priority: Major
>              Labels: posix
>
> * POSIX scripts should not use bash-specific features
>  * Fix suggestions from shellcheck 
> ([https://github.com/apache/maven-wrapper/pull/74])



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to