Skip to content

Commit

Permalink
Repair acceptance test (#1584)
Browse files Browse the repository at this point in the history
* Corrected a problem with the acceptance test action.
It seems the replacement for ::set-output (i.e. >> $GITHUB_OUTPUT) does not accept newlines in the data.

* Removed some quotes and an extra parenthesis.
  • Loading branch information
jcpitre committed Sep 19, 2023
1 parent 528ac92 commit fa1dbcf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/acceptance_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
# Obtain the last commit from the branch to merge (hence the HEAD^2).
# In case of multi-line commit messages, remove any \n, because the GITHUB_OUTPUT method
# of sending data to other jobs does not like them.
run: echo "pr_commit_message=\"$(git log --format=%B -n 1 HEAD^2 | tr '\n' ' '))\"" >> $GITHUB_OUTPUT
run: echo "pr_commit_message=$(git log --format=%B -n 1 HEAD^2 | tr '\n' ' ')" >> $GITHUB_OUTPUT

# For **Pull Request** events this will resolve to something like "$( [ -z "commit message pr" ] && echo "" || echo "commit message pr" )" which then resolves to just "commit message pr"
outputs:
Expand Down

0 comments on commit fa1dbcf

Please sign in to comment.