Skip to content

Commit f5e2acc

Browse files
authored
Merge pull request #436 from ExpediaGroup/add-missing-input
fix: forgot to add new input to action.yml
2 parents c10fad6 + 8fd1d82 commit f5e2acc

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

.github/workflows/test.yaml

+7-3
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,14 @@ jobs:
2424
- name: Check for diff
2525
id: diff
2626
run: |
27-
git status -s dist/ | grep "dist/"
28-
echo "GIT_STATUS=$?" >> "$GITHUB_OUTPUT"
27+
if git status -s dist/ | grep "dist/"; then
28+
echo "SHOULD_PUSH_BUILD=true" >> "$GITHUB_OUTPUT"
29+
else
30+
echo "SHOULD_PUSH_BUILD=false" >> "$GITHUB_OUTPUT"
31+
fi
32+
2933
- name: Push build
30-
if: ${{ steps.diff.outputs.GIT_STATUS == '0' }}
34+
if: ${{ steps.diff.outputs.SHOULD_PUSH_BUILD == 'true' }}
3135
run: |
3236
git config user.name github-actions
3337
git config user.email [email protected]

action.yml

+3
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@ inputs:
1616
description: AWS Region
1717
required: false
1818
default: us-west-2
19+
github_token:
20+
description: Repo token for fetching commit details
21+
required: false
1922
runs:
2023
using: "node16"
2124
main: "dist/index.js"

0 commit comments

Comments
 (0)