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 22 #166430

Merged
merged 19 commits into from
Feb 28, 2023
Merged

chore: update to Electron 22 #166430

merged 19 commits into from
Feb 28, 2023

Conversation

deepak1556
Copy link
Collaborator

@deepak1556 deepak1556 commented Nov 16, 2022

@deepak1556 deepak1556 self-assigned this Nov 16, 2022
@deepak1556 deepak1556 added this to the November 2022 milestone Nov 16, 2022
@deepak1556
Copy link
Collaborator Author

This is blocked on Electron missing https://source.chromium.org/chromium/chromium/src/+/main:buildtools/third_party/libc++/trunk/include/__memory_resource/; headers from the libcxx bundle shipped.

@deepak1556 deepak1556 force-pushed the electron-22.x.y branch 2 times, most recently from 5efaccc to 9718242 Compare December 21, 2022 13:08
@deepak1556 deepak1556 mentioned this pull request Jan 16, 2023
7 tasks
@deepak1556 deepak1556 force-pushed the electron-22.x.y branch 3 times, most recently from 5d948e6 to fe17055 Compare January 23, 2023 15:11
@deepak1556 deepak1556 force-pushed the electron-22.x.y branch 3 times, most recently from 3137bee to 085fa33 Compare January 27, 2023 02:51
@deepak1556 deepak1556 force-pushed the electron-22.x.y branch 4 times, most recently from 7b2de25 to c653027 Compare February 10, 2023 04:14
@deepak1556 deepak1556 force-pushed the electron-22.x.y branch 5 times, most recently from 9aa94f9 to f70db26 Compare February 16, 2023 09:03
@@ -91,7 +91,7 @@ function darwinBundleDocumentTypes(types: { [name: string]: string | string[] },
}

export const config = {
version: product.electronRepository ? '19.1.11' : util.getElectronVersion(),
version: product.electronRepository ? '22.10.0' : util.getElectronVersion(),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I will update this to an official version once the PGO related builds are available in a follow-up PR. For now, lets merge with the custom version to get insiders coverage.

@deepak1556 deepak1556 marked this pull request as ready for review February 28, 2023 12:22
@deepak1556 deepak1556 requested a review from bpasero February 28, 2023 12:22
@deepak1556
Copy link
Collaborator Author

I will update distro once this is merged.

@deepak1556 deepak1556 merged commit 6e85d2b into main Feb 28, 2023
@deepak1556 deepak1556 deleted the electron-22.x.y branch February 28, 2023 15:04
deepak1556 added a commit that referenced this pull request Mar 2, 2023
deepak1556 added a commit that referenced this pull request Mar 2, 2023
* Revert "Update to min supported version of node (#175723)"

This reverts commit f749e6b.

* Revert "chore: update min node version for development (#175663)"

This reverts commit f2a822f.

* Revert "electron 22 sandbox - enable for OSS (#175659)"

This reverts commit 2551af2.

* Revert "chore: update to Electron 22 (#166430)"

This reverts commit 6e85d2b.
@samuelstroschein
Copy link

Has this PR been reverted?

@deepak1556
Copy link
Collaborator Author

@samuelstroschein yes in favor of #175843, we will post more context in the following week.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.