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

Update to Electron 9 #100731

Closed
deepak1556 opened this issue Jun 22, 2020 · 6 comments
Closed

Update to Electron 9 #100731

deepak1556 opened this issue Jun 22, 2020 · 6 comments
Assignees
Labels
insiders-released Patch has been released in VS Code Insiders plan-item VS Code - planned item for upcoming
Milestone

Comments

@deepak1556
Copy link
Collaborator

deepak1556 commented Jun 22, 2020

This plan item is to cover the work of updating to Electron 9.

Current Status:

  • Update merged to master as version 9.1.0
  • Insiders testing this week
@deepak1556 deepak1556 added plan-item VS Code - planned item for upcoming electron-8-update labels Jun 22, 2020
@deepak1556 deepak1556 added this to the June 2020 milestone Jun 22, 2020
@deepak1556 deepak1556 self-assigned this Jun 22, 2020
@deepak1556 deepak1556 modified the milestones: June 2020, July 2020 Jun 29, 2020
@tduyduc
Copy link

tduyduc commented Jul 6, 2020

Please note that this Electron update will update Chrome to version 79 or above. Chrome with versions from 79 is known to cause problems with some input methods for Linux.
There are bug reports about this matter here, here, and here, and none of them have been resolved yet.

@deepak1556 deepak1556 changed the title Update to Electron 8 Update to Electron 8/9 Jul 11, 2020
@deepak1556
Copy link
Collaborator Author

We have found electron 8 has quite a few regression wrt chromium devtools and with vscode also disabling LayoutNG, this is causing some unknown renderer freezes or layout artifacts in electron 8. Both of these issues are fixes in Electron 9 which bumps chromium version to 83. We are exploring to update to this version in July iteration.

@PF4Public
Copy link

PF4Public commented Aug 3, 2020

Why was "chore: update to electron 9-x-y" reverted recently?
Is it not coming soon?

@nrayburn-tech
Copy link
Contributor

@PF4Public Some more information here, #103060 (comment).

@deepak1556
Copy link
Collaborator Author

We are moving forward with Electron 9 for stable this milestone.

@deepak1556 deepak1556 added the insiders-released Patch has been released in VS Code Insiders label Aug 28, 2020
@ghiscoding
Copy link

It seems that the push to latest version of Electron adds problem with Cypress, I'm getting the following error with VSCode Insiders (currently 1.50.0) since the last 1-2 versions of Insiders when trying to open Cypress from the VSCode terminal on Win10

electron: -max-http-header-size=1048576 is not allowed in NODE_OPTIONS

This works totally fine with VSCode Stable.

There's this Cypress issue that was opened but later closed by the user because it was suggested to start Cypress from a regular Windows terminal to start Cypress, while that works from a regular terminal, I would rather start it from VSCode since I have npm script shortcuts to start it from there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
insiders-released Patch has been released in VS Code Insiders plan-item VS Code - planned item for upcoming
Projects
None yet
Development

No branches or pull requests

5 participants