Skip to content

Commit

Permalink
Merge pull request #5380 from nextcloud/ci/update-node-dist
Browse files Browse the repository at this point in the history
fix(ci): use output rather than failure
  • Loading branch information
max-nextcloud authored Feb 12, 2024
2 parents 2a4cf7e + 0feb769 commit 470fc70
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/update-node-dist.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,14 @@ jobs:
id: changes
continue-on-error: true
run: |
bash -c "[[ ! \"`git status --porcelain `\" ]] || (echo 'Please recompile and commit the assets, see the section \"Show changes on failure\" for details' && exit 1)"
{
echo 'CHANGED<<EOF'
git status --porcelain
echo 'EOF'
} >> "$GITHUB_OUTPUT"
- name: Add and commit
if: steps.changes.outcome == 'failure'
if: steps.changes.outputs.CHANGED != ''
run: |
git add --force js/
git commit --signoff -m 'chore(assets): recompile assets'
Expand Down

0 comments on commit 470fc70

Please sign in to comment.