Skip to content

Commit

Permalink
Only get the subject of the commit message for notifications (#3689)
Browse files Browse the repository at this point in the history
When commit messages are not properly cleaned up during a squash and merge they could have characters that make the workflow break.

With this change we're only retrieving the subject of the commit message (instead of the whole body) reducing the probability of breaking the workflow.
  • Loading branch information
lucacome authored Mar 28, 2023
1 parent d11e3bc commit 399f52f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/updates-notification.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ jobs:
- name: Get variables for Slack
id: slack
run: |
echo "message=$(git log -1 --pretty=%B)" >> $GITHUB_OUTPUT
echo "message=$(git log -1 --pretty=%s)" >> $GITHUB_OUTPUT
echo "date=$(date +%s)" >> $GITHUB_OUTPUT
echo "sha_short=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
- name: Send Notification
Expand Down

0 comments on commit 399f52f

Please sign in to comment.