Skip to content

Commit

Permalink
chore: Reduced lost pixel complexity (#78)
Browse files Browse the repository at this point in the history
  • Loading branch information
nickevansuk authored Aug 9, 2024
1 parent dc2e11b commit 991f3f9
Show file tree
Hide file tree
Showing 2 changed files with 55 additions and 63 deletions.
56 changes: 55 additions & 1 deletion .github/workflows/test-and-deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,8 @@ jobs:

test-site:
runs-on: ubuntu-latest
outputs:
lost-pixel-conclusion: ${{ steps.lp.conclusion }}
steps:
- name: Checkout validator site
uses: actions/checkout@v2
Expand All @@ -84,7 +86,59 @@ jobs:
with:
name: lost-pixel-artefacts
path: .lostpixel


create-pr-to-update-lost-pixel-baseline:
runs-on: ubuntu-latest
needs: [test-site, test-models, test-data-model-validator, test-rpde-validator]
# Only run this job on branches other than master; this is to prevent the Lost Pixel baseline from being updated on the master branch.
# The baseline should only be updated on PRs, not on the master branch.
if: ${{ failure() && github.ref != 'refs/heads/master' && needs.test-site.outputs.lost-pixel-conclusion == 'failure'}}
steps:
- name: Checkout validator site
uses: actions/checkout@v2
- name: Use Node.js 14.x
uses: actions/setup-node@v1
with:
node-version: 14
- name: Install
run: npm install
- name: Test
run: npm test
- name: Start server (in background)
run: npm run start &

- name: Lost Pixel
uses: lost-pixel/[email protected]
env:
LOST_PIXEL_MODE: update

# Note this deletes the branch and PR if differences are resolved
- name: Create Pull Request
id: cpr
uses: peter-evans/create-pull-request@v6
with:
path: ./
token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }}
commit-message: Update lost-pixel baselines
committer: openactive-bot <[email protected]>
author: openactive-bot <[email protected]>
signoff: false
branch: 'lost-pixel-update/${{ github.ref_name }}'
delete-branch: true
title: 'Lost Pixel update - ${{ github.ref_name }}'
body: |
Automated baseline update PR created by Lost Pixel
Please check the updated baseline images are as expected before merging this PR.
labels: |
automated pr
draft: false

- name: Check outputs
run: |
echo "Pull Request Number - ${{ steps.cpr.outputs.pull-request-number }}"
echo "Pull Request URL - ${{ steps.cpr.outputs.pull-request-url }}"
deploy-site:
needs: [test-site, test-models, test-data-model-validator, test-rpde-validator]
if: ${{ github.ref == 'refs/heads/master' }}
Expand Down
62 changes: 0 additions & 62 deletions .github/workflows/update-lost-pixel-baseline.yml

This file was deleted.

0 comments on commit 991f3f9

Please sign in to comment.