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 to electron 30 #215406

Merged
merged 14 commits into from
Jul 9, 2024
Merged

chore: update to electron 30 #215406

merged 14 commits into from
Jul 9, 2024

Conversation

deepak1556
Copy link
Contributor

@deepak1556 deepak1556 commented Jun 13, 2024

Fixes #214394
Fixes #215924

@deepak1556 deepak1556 added this to the June 2024 milestone Jun 13, 2024
@deepak1556 deepak1556 self-assigned this Jun 13, 2024
@deepak1556 deepak1556 force-pushed the electron-30.x.y branch 5 times, most recently from cdbcba3 to a8f7d1c Compare June 21, 2024 07:38
@deepak1556 deepak1556 force-pushed the electron-30.x.y branch 3 times, most recently from 874284c to 375dcbf Compare June 26, 2024 01:45
@deepak1556 deepak1556 modified the milestones: June 2024, July 2024 Jun 28, 2024
@deepak1556 deepak1556 force-pushed the electron-30.x.y branch 2 times, most recently from 82a145f to 31bd299 Compare July 3, 2024 08:33
@deepak1556 deepak1556 force-pushed the electron-30.x.y branch 3 times, most recently from faa6fd6 to 33083e5 Compare July 8, 2024 00:14
@deepak1556
Copy link
Contributor Author

@chrmarti please review e4dd212

@connor4312 please review 6c1649e

@bpasero for rest of the files.

@@ -16,7 +16,7 @@ steps:
inputs:
versionSource: fromFile
versionFilePath: .nvmrc
nodejsMirror: https://github.com/joaomoreno/node-mirror/releases/download
Copy link
Contributor Author

@deepak1556 deepak1556 Jul 8, 2024

Choose a reason for hiding this comment

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

@deepak1556 deepak1556 marked this pull request as ready for review July 8, 2024 00:29
@connor4312
Copy link
Member

lgtm

bpasero
bpasero previously approved these changes Jul 8, 2024
chrmarti
chrmarti previously approved these changes Jul 8, 2024
Copy link
Contributor

@chrmarti chrmarti left a comment

Choose a reason for hiding this comment

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

LGTM!

@deepak1556 deepak1556 dismissed stale reviews from chrmarti and bpasero via 19fa521 July 8, 2024 07:02
bpasero
bpasero previously approved these changes Jul 8, 2024
@deepak1556
Copy link
Contributor Author

PR is ready for merge, but I will merge it tomorrow since the new dependency has not been cached yet by the registry mirror.

@deepak1556 deepak1556 merged commit 2fe0527 into main Jul 9, 2024
40 checks passed
@deepak1556 deepak1556 deleted the electron-30.x.y branch July 9, 2024 05:33
aaronchucarroll pushed a commit to aaronchucarroll/vscode that referenced this pull request Jul 10, 2024
* chore: update electron@30.0.9

* chore: update rpm deps

* chore: bump electron@30.1.2

* fix: update kerberos for Node.js 20.x

Refs mongodb-js/kerberos@c1f7aca

* fix: use shell when spawning .bat or .cmd files

Refs nodejs/node@6627222409

* fix: update @vscode/test-electron@2.4.0

Refs microsoft/vscode-test@3f7a3cc

* fixup! use shell when spawning .bat or .cmd files

* chore: bump nodejs@20.14.0 internal build

* ci: skip nodejsMirror for 20.14.0 due to missing builds

* fixup! use shell when spawning .bat or .cmd files

* chore: update debian deps

* fixup! skip nodejsMirror for 20.14.0 due to missing builds

* fix: universal build

- Updates vscode-universal-bundler to support x64ArchFiles option
- Kerberos starts building universal binaries which should now be
  skipped from lipo step via x64ArchFiles
- Skips bundling *.mk files

* chore: bump distro
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.

Code 1.90.1 crashes on startup Electron 30 update
4 participants