Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Upgrade electron builder to latest version #629

Merged
merged 4 commits into from
Dec 2, 2021

Conversation

achou11
Copy link
Member

@achou11 achou11 commented Nov 22, 2021

Fixes #625 (see builder changelog)

Tested by successfully installing the associated MacOS zip asset generated in CI

Notes:

  • Important: Upgrades the node version used in development from 12 to 14, which is required for this dependency upgrade. This does not affect the node runtime that's used within Electron itself, which still remains on 12 in this case

@@ -71,7 +71,7 @@ jobs:
MAPEO_VARIANT: ${{ matrix.variant }}
- name: Cleanup artifacts
run: |
npx rimraf@2 'dist/${{ matrix.variant }}/!(*.exe|*.deb|*.AppImage|*.dmg|*.yml|github)'
npx rimraf@2 'dist/${{ matrix.variant }}/!(*.exe|*.deb|*.AppImage|*.dmg|*.yml|*.zip|github)'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

initially just changed this so I could actually download the asset from CI for testing this change, but maybe it's just worth keeping anyways

uses: actions/setup-node@v2
with:
node-version: 12.14.1
node-version: 14.18
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Think it should be okay to not be specific about patch version but happy to change if necessary

@achou11 achou11 force-pushed the ac/625/zip-macos-fix branch from 66ab88a to 852c040 Compare December 2, 2021 17:30
@achou11 achou11 merged commit 6f76531 into master Dec 2, 2021
@achou11 achou11 deleted the ac/625/zip-macos-fix branch December 2, 2021 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't start app from ZIP on MacOS
1 participant