Skip to content

Commit

Permalink
Merge pull request #854 from carstingaxion/fix/screenshot-workflow-tr…
Browse files Browse the repository at this point in the history
…igger-image-compression-3

Minor fixes
  • Loading branch information
carstingaxion authored Sep 4, 2024
2 parents b97f4b8 + 8dc487d commit 8e35a60
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/wordpress-org-screenshots.yml
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ jobs:
git restore ./build
git add -A
git commit -m "Screenshots for ${{ matrix.locale }} updated!"
git push origin fix/wp-org-screenshots-${{ matrix.locale }}-${{ github.sha }}
# git push origin fix/wp-org-screenshots-${{ matrix.locale }}-${{ github.sha }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Expand All @@ -180,7 +180,7 @@ jobs:

# The compress images workflow says: "This action only runs for pushes to PRs"
# So lets TRY to put the push event after the pr creation,
# to hopefully get this triggerd now.
# to hopefully get this triggered now.
- name: Push updated screenshots
# Using "continue-on-error:true" results in 'conclusion' being a success in any case, while the 'outcome' can differ.
# And, as you already guessed, if: failure() looks at 'conclusion'.
Expand Down

0 comments on commit 8e35a60

Please sign in to comment.