Skip to content

Commit

Permalink
"actions/upload-artifact" macOS package with electron@26
Browse files Browse the repository at this point in the history
  • Loading branch information
vladimiry committed Nov 17, 2023
1 parent c5f76ec commit 2a7acdb
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# - https://github.com/actions/runner/issues/646

name: "GitHub Actions CI"
on: { push: { branches: [ master ] }, pull_request: { branches: [ master ] } }
on: { push: { branches: [ macos-downgraded-electron ] }, pull_request: { branches: [ master ] } }
env: { ELECTRON_MAIL_NODE_VERSION: 18, PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1 }
jobs:
build-proton-clients:
Expand Down Expand Up @@ -35,7 +35,7 @@ jobs:
- { name: tar artifact, run: tar -cvf proton-clients-artifact.tar ./output/git/backup ./app/assets/db-search-monaco-editor }
- { name: upload artifact, uses: actions/upload-artifact@v2, with: { name: proton-clients-artifact, path: ./proton-clients-artifact.tar, if-no-files-found: error } }
build-app:
strategy: { matrix: { os: [ ubuntu-20.04, windows-2019, macos-11 ] }, fail-fast: false }
strategy: { matrix: { os: [ macos-11 ] }, fail-fast: false }
runs-on: ${{ matrix.os }}
steps:
# <common> TODO use YAML anchors feature when it gets supported
Expand All @@ -49,13 +49,14 @@ jobs:
- { name: envinfo, run: npx envinfo }
- { name: install node modules, run: "pnpm install --frozen-lockfile --reporter=append-only" }
# </common>
- { name: build, run: 'pnpm add -D electron@26' }
- { name: build, run: 'npm exec --package=npm-run-all -- npm-run-all lint build' }
- { name: print ./app files, shell: bash, run: ./scripts/ci/list-fs-content.sh ./app }
- { name: upload artifact, uses: actions/upload-artifact@v2, with: { name: 'app-artifact-${{ runner.os }}', path: ./app, if-no-files-found: error } }
package-app:
if: github.event_name == 'push'
needs: [ build-proton-clients, build-app ]
strategy: { matrix: { os: [ ubuntu-20.04, windows-2019, macos-11, macos-latest ] }, fail-fast: false }
strategy: { matrix: { os: [ macos-11 ] }, fail-fast: false }
runs-on: ${{ matrix.os }}
steps:
# <common> TODO use YAML anchors feature when it gets supported
Expand All @@ -69,6 +70,7 @@ jobs:
- { name: envinfo, run: npx envinfo }
- { name: install node modules, run: "pnpm install --frozen-lockfile --reporter=append-only" }
# </common>
- { name: build, run: 'pnpm add -D electron@26' }
- { name: download proton clients artifact, uses: actions/download-artifact@v2, with: { name: proton-clients-artifact } }
- { name: unpack proton clients artifact, shell: bash, run: tar -xvf ./proton-clients-artifact.tar }
- { name: download app artifact, uses: actions/download-artifact@v2, with: { name: 'app-artifact-${{ runner.os }}', path: ./app } }
Expand All @@ -77,6 +79,7 @@ jobs:
- { name: build, if: runner.os == 'Windows', run: ./scripts/ci/github/package-app.ps1 }
- { name: build, if: runner.os == 'Linux', run: ./scripts/ci/github/package-app-linux.sh }
- { name: build, if: matrix.os == 'macos-11', run: ./scripts/ci/github/package-app-osx.sh, env: { MACOSX_DEPLOYMENT_TARGET: "10.10" } }
- { name: upload app package, if: matrix.os == 'macos-11', uses: actions/upload-artifact@v3, with: { name: 'app-package-dmg-${{ matrix.os }}', path: ./dist/*.dmg, if-no-files-found: error } }
- { name: build, if: matrix.os == 'macos-latest', run: ./scripts/ci/github/package-app-osx.sh, env: { MACOSX_DEPLOYMENT_TARGET: "11.0", ELECTRON_DEST_MAIL_ARCH: "arm64" } }
- name: upload e2e/spectron output
if: ${{ always() }}
Expand Down
2 changes: 1 addition & 1 deletion scripts/ci/github/package-app-osx.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,5 +28,5 @@ echo "::endgroup::"

echo "::group::hash & upload"
pnpm run scripts/dist-packages/print-hashes
pnpm run scripts/dist-packages/upload
# pnpm run scripts/dist-packages/upload
echo "::endgroup::"

0 comments on commit 2a7acdb

Please sign in to comment.