Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Commit

Permalink
Fix Maven command on Windows
Browse files Browse the repository at this point in the history
Was considered invalid because "$INPUT_MAVEN_ARGS" was not recognized as an environment variable
  • Loading branch information
samuelmeuli committed Dec 16, 2019
1 parent a1a11bf commit c2db302
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 0 additions & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ inputs:
maven_args:
description: Additional flags/arguments to pass to the Maven deploy command
required: false
default: ""

runs:
using: node12
Expand Down
4 changes: 3 additions & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,9 @@ const runAction = () => {
// deployment and not in the install phase
log("Deploying the Maven project…");
run(
`mvn clean deploy --batch-mode --activate-profiles deploy --settings ${mavenSettingsPath} $INPUT_MAVEN_ARGS`,
`mvn clean deploy --batch-mode --activate-profiles deploy --settings ${mavenSettingsPath} ${getInput(
"maven_args",
) || ""}`,
getInput("directory"),
);
};
Expand Down

0 comments on commit c2db302

Please sign in to comment.