Skip to content

Commit 484575d

Browse files
committed
minor #2687 [CI] Use git diff instead of git status (smnandre)
This PR was merged into the 2.x branch. Discussion ---------- [CI] Use git diff instead of git status The CI often fails on the "check dist files" script because some dist file changed .. with no real content change This PR replace the `git status --porcelain` with a `git diff --quiet` to ignore this situation. Commits ------- 11114f3 [CI] Use git diff instead of git status
2 parents af16ce1 + 11114f3 commit 484575d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/test.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141

4242
- name: Check if JS dist files are current
4343
run: |
44-
if [[ -n $(git status --porcelain) ]]; then
44+
if ! git diff --quiet; then
4545
echo "The Git workspace is unclean! Changes detected:"
4646
git status --porcelain
4747
git diff

0 commit comments

Comments
 (0)