-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
feat(deps): update dependency electron to v15 🌟 #18317
Conversation
See the guidelines for reviewing dependency updates for info on how to review dependency update PRs. |
Test summaryRun details
View run in Cypress Dashboard ➡️ FailuresThis comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
35fc07b
to
f786f9a
Compare
f786f9a
to
a11ccbd
Compare
ff0cbab
to
4cc5681
Compare
rationale: without setting `proxy`, `destroyed` is still false after upgrading to node 15 - failure has nothing to do with our proxying.
30f24a0
to
8bf95db
Compare
This reverts commit bb002b5.
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.
Looks good to me, outside of all the failing tests 😅
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
BREAKING CHANGE: Updates Node major version from 14 to 16. No other breakage.
User-facing changelog
Upgraded Electron to 15.2.0. Bumps bundled Node to 16.5.0 and Chromium to 94.0.4606.71
Notes:
autoDestroy
http: useautoDestroy: true
in incoming message nodejs/node#33035autoDestroy
net: autoDestroy Socket nodejs/node#31806'aborted'
IncomingMessage should'error'
nodejs/node#33172@cypress/request
: fix: treat aborts the same as before node 16 request#7net/tls.connect
with no options is no longer supported. We only do this in tests. net: handle net.connect() without arguments nodejs/node#34022main
does not exist. Movedmain
tofiles
for packages where it DNE at dev time.install-required-node
step was adapted Unable to use v12 with node v15+ ("engines.node" field misuse) angular/angular#42076 (comment)