-
Notifications
You must be signed in to change notification settings - Fork 29.9k
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 18 #145527
chore: update to electron 18 #145527
Conversation
8aad31b
to
6e2b28b
Compare
Electron 18.0.1 has been released, so this PR could use electron 18.0.1 instead. |
It will be updated when corresponding internal builds are available. |
6e2b28b
to
e3c4141
Compare
Electron 18.2.0 is stable enough |
Please bump to electron 19.0.1 straight. Electron 19 switched to Chromium 102 which contains an important fix to make it run on Linux on Apples M1 chips, see: https://github.com/AsahiLinux/docs/wiki/Broken-Software |
e3c4141
to
47de954
Compare
@mkurz can you point to the upstream bug that addresses the issue ? We can try backporting the fix to Electron 18 if it is possible. Just to give some context on Electron version adoption for VSCode, we currently follow an 8 week stabilization cycle, the first 4 weeks we have our nightly exploration builds that is largely self-hosted by our team and then we have an endgame testing using these builds. Once we are confident with exploration builds the update would then be pushed to insiders in the following iteration for wider testing from the VSCode community and it may or may not be pushed to stable depending on the data we acquire from insiders testing. Since we have already spent time stabilizing Electron 18 for exploration builds this iteration, a jump to Electron 19 at this point would only delay the next update adoption. On the other hand, once Electron 18 is promoted to insiders we can actively look into adopting Electron 19 for exploration builds. |
47de954
to
0f2f8d1
Compare
https://bugs.chromium.org/p/chromium/issues/detail?id=1301788 |
Change has been backported at electron/electron#34385 |
Thanks @deepak1556! |
ca6743a
to
e1720f9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏
Fixes #144962
Refs #146584
Fixes #108509