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

Upgrade to Electron 7 #8959

Closed
wants to merge 29 commits into from
Closed

Upgrade to Electron 7 #8959

wants to merge 29 commits into from

Conversation

kuychaco
Copy link
Contributor

@kuychaco kuychaco commented Jan 22, 2020

⚠️Superseded by #8967 ⚠️

Description

This PR takes us from Electron v6 to v7. Note that #8791 contains the work that takes us from Electron v5 (currently on stable) to v6. I'm keeping these PRs separate to make it easier to review the change-set associated with each version bump.

If all looks good with Electron v7 we'll ship it. If not, we'll fall back to shipping with Electron v6.

Bugs to test / keep in mind

Release notes

Notes:

kuychaco and others added 28 commits December 12, 2019 14:36
Co-Authored-By: Josh Abernathy <joshaber@gmail.com>
Co-Authored-By: Josh Abernathy <joshaber@gmail.com>
Co-Authored-By: Josh Abernathy <joshaber@gmail.com>
Co-Authored-By: evelyn masso <outofambit@github.com>
Co-Authored-By: evelyn masso <outofambit@github.com>
Co-Authored-By: evelyn masso <outofambit@github.com>
Commit box melts down off the screen

Co-Authored-By: evelyn masso <outofambit@github.com>
doesn't jive with our new deps
i think i fixed this incorrectly for the electron 5 upgrade. hopefully this is more correct.
This replaces the patch for osx-notarize removed in 4898ffb, which was causing the build to fail

See electron/packager#1069

Co-Authored-By: evelyn masso <outofambit@github.com>
node_modules/.bin/prettier --write app/src/ui/app.tsx
Co-Authored-By: evelyn masso <outofambit@github.com>
Co-Authored-By: evelyn masso <outofambit@github.com>
Co-Authored-By: evelyn masso <outofambit@github.com>
Co-Authored-By: evelyn masso <outofambit@github.com>
@kuychaco kuychaco changed the base branch from ku-electron-6-upgrade to development January 23, 2020 19:36
@kuychaco kuychaco added this to In Progress PRs in Desktop 2.3 release via automation Jan 27, 2020
@kuychaco kuychaco moved this from In Progress PRs to Awaiting Review in Desktop 2.3 release Jan 27, 2020
@kuychaco kuychaco changed the base branch from development to ku-electron-6-upgrade January 29, 2020 03:13
@kuychaco kuychaco changed the base branch from ku-electron-6-upgrade to development January 29, 2020 03:13
@kuychaco kuychaco changed the title Upgrade from Electron v6 to v7 Upgrade to Electron 7 Jan 29, 2020
@kuychaco
Copy link
Contributor Author

kuychaco commented Feb 3, 2020

Closing this as it's been superseded by #8967

@kuychaco kuychaco closed this Feb 3, 2020
@outofambit outofambit deleted the ku-electron-7-upgrade branch April 21, 2020 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Desktop 2.3 release
Awaiting Review
Development

Successfully merging this pull request may close these issues.

None yet

3 participants