screenshot comparison: prepare macos folder in the expected format by upload-artifact action (#15911)
# Objective - After #15894, creenshot upload now works on linux but still fails on macOS - Artifact with screenshot is not prepared correctly:  ## Solution - Try to prepare it like the `upload-artifact` action expects it
This commit is contained in:
parent
e79bc7811d
commit
8c0fcf02d0
12
.github/workflows/ci-comment-failures.yml
vendored
12
.github/workflows/ci-comment-failures.yml
vendored
@ -208,12 +208,16 @@ jobs:
|
|||||||
var fs = require('fs');
|
var fs = require('fs');
|
||||||
fs.writeFileSync('${{github.workspace}}/screenshots-macos.zip', Buffer.from(download.data));
|
fs.writeFileSync('${{github.workspace}}/screenshots-macos.zip', Buffer.from(download.data));
|
||||||
return "true"
|
return "true"
|
||||||
- run: unzip screenshots-macos.zip
|
- name: prepare artifact folder
|
||||||
|
run: |
|
||||||
|
unzip screenshots-macos.zip
|
||||||
|
mkdir screenshots
|
||||||
|
mv screenshots-* screenshots/
|
||||||
- name: save screenshots
|
- name: save screenshots
|
||||||
uses: actions/upload-artifact@v4
|
uses: actions/upload-artifact@v4
|
||||||
with:
|
with:
|
||||||
name: screenshots-macos
|
name: screenshots-macos
|
||||||
path: screenshots-macos
|
path: screenshots
|
||||||
|
|
||||||
|
|
||||||
compare-macos-screenshots:
|
compare-macos-screenshots:
|
||||||
@ -221,8 +225,8 @@ jobs:
|
|||||||
needs: [make-macos-screenshots-available]
|
needs: [make-macos-screenshots-available]
|
||||||
uses: ./.github/workflows/send-screenshots-to-pixeleagle.yml
|
uses: ./.github/workflows/send-screenshots-to-pixeleagle.yml
|
||||||
with:
|
with:
|
||||||
commit: ${{ github.event.workflow_run.event.sha }}
|
commit: ${{ github.event.workflow_run.head_commit }}
|
||||||
branch: ${{ github.event.workflow_run.event.ref_name }}
|
branch: ${{ github.event.workflow_run.head_branch }}
|
||||||
artifact: screenshots-macos
|
artifact: screenshots-macos
|
||||||
os: macos
|
os: macos
|
||||||
secrets: inherit
|
secrets: inherit
|
||||||
|
Loading…
Reference in New Issue
Block a user