Skip to content

Commit

Permalink
build: upgrade visual snapshot to support new artifacts api (#3399)
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.


ref: linz/basemaps-config#1039
  • Loading branch information
blacha authored Feb 11, 2025
1 parent 72f2a81 commit ef3a85e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/screenshot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
docker run --rm --network="host" -v $PWD:$PWD ghcr.io/linz/basemaps-screenshot/cli:v1 --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 All @@ -68,7 +68,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 ef3a85e

Please sign in to comment.