Skip to content

Commit

Permalink
build: upgrade visual snapshot to support new artifacts api (#1039)
Browse files Browse the repository at this point in the history
### Motivation

`getsentry/action-visual-snapshot` has been deprecated and archived, it
is no longer supported, github has recently removed support for the
artifacts API it was using to store the images, causing all of these
actions to fail.

### Modifications

Fork the github action and update it to support the newer github actions
artifacts API.

this is a temporary fix with the intention to replace this github action
with another more supported action in the near term.

### Verification

Checking to see if the build succeeds after this is merged.
  • Loading branch information
blacha authored Feb 11, 2025
1 parent eb0e73f commit 31b703c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ jobs:
docker run --rm --network="host" -v $PWD:$PWD ${DOCKER_AWS_ENV} ${SCREENSHOT_CONTAINER} --url http://localhost:5000 --output $PWD/.artifacts/visual-snapshots
- name: Save snapshots
uses: getsentry/action-visual-snapshot@v2
uses: blacha/action-visual-snapshot@v2-next
with:
save-only: true
snapshot-path: .artifacts/visual-snapshots
Expand Down Expand Up @@ -284,7 +284,7 @@ jobs:

- name: Diff snapshots
id: visual-snapshots-diff
uses: blacha/action-visual-snapshot@v2
uses: blacha/action-visual-snapshot@v2-next
with:
storage-prefix: 's3://linz-basemaps-screenshot'
storage-url: 'https://d25mfjh9syaxsr.cloudfront.net'

0 comments on commit 31b703c

Please sign in to comment.