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

chore: Update electron and related dependencies #7675

Merged

Conversation

marc2332
Copy link
Collaborator

Closes #7655

@marc2332 marc2332 marked this pull request as ready for review November 2, 2023 11:50
patches/electron-updater+5.3.0.patch Show resolved Hide resolved
"electron-builder": "^24.0.0",
"electron-notarize": "^1.0.0",
"electron": "27.0.2",
"electron-builder": "^24.6.4",
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if this was in previous versions of electron-builder, but we may need to adjust our electron-builder-config.js so that we don't notarize the app twice (it's unclear what the default behavior of the notarize option is — it seems to wrap around @electron/notarize)

Copy link
Collaborator Author

@marc2332 marc2332 Jan 29, 2024

Choose a reason for hiding this comment

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

We weren't signing twice. Anyway, I have now changed a bit the config, electron-builder will now sign for us instead of us manually calling @electron/notarize

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@marc2332 marc2332 force-pushed the chore/update-electron branch from c1b7510 to 632ee34 Compare December 20, 2023 10:05
@begonaalvarezd begonaalvarezd self-requested a review January 15, 2024 11:53
@begonaalvarezd begonaalvarezd changed the base branch from develop to release/desktop-shimmer-2.1.11 February 1, 2024 10:45
@begonaalvarezd begonaalvarezd merged commit 0dcbb20 into release/desktop-shimmer-2.1.11 Feb 1, 2024
4 checks passed
@begonaalvarezd begonaalvarezd deleted the chore/update-electron branch February 1, 2024 11:15
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.

[Task]: Update electron to max possible
3 participants