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(dependency): Upgrade Electron 28 #28959
base: develop
Are you sure you want to change the base?
Conversation
5 failed and 5 flaky tests on run #54423 ↗︎
Details:
cypress\e2e\runner\reporter.errors.cy.ts • 2 failed tests • app-e2e
cypress\e2e\studio\studio.cy.ts • 2 failed tests • app-e2e
cypress\e2e\runner\reporter.hooks.cy.ts • 1 failed test • app-e2e
cypress\e2e\create-from-component.cy.ts • 1 flaky test • app-e2e
cypress\e2e\specs.cy.ts • 1 flaky test • app-e2e
cypress\e2e\top-nav.cy.ts • 1 flaky test • app-e2e
cypress\e2e\specs_list_latest_runs.cy.ts • 1 flaky test • app-e2e
cypress\e2e\migration.cy.ts • 1 flaky test • launchpad-e2e
Review all test suite changes for PR #28959 ↗︎ |
…nto electron-28
@@ -213,7 +213,7 @@ class SourcemapSupport { | |||
frame = cloneCallSite(frame) | |||
|
|||
frame.getFileName = function getFileName () { | |||
return pos.source || pos.name || null | |||
return pos.source || pos.name || undefined |
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.
This type check was failing, likely due to upgrading types/node. https://app.circleci.com/pipelines/github/cypress-io/cypress/59945/workflows/36318f1d-bf0e-42e7-b0bc-3fa50a5b29f5/jobs/2493950
yarn.lock
Outdated
integrity sha512-2s6B2CWZM//kPgwnuI0KrYwNjfdByE25zvAaEpq9IH4zcNsarH8Ihu/UuX6XMPEogDAxkuUFeZn60pXNHAqn3A== | ||
version "3.54.0" | ||
resolved "https://registry.yarnpkg.com/node-abi/-/node-abi-3.54.0.tgz#f6386f7548817acac6434c6cba02999c9aebcc69" | ||
integrity sha512-p7eGEiQil0YUV3ItH4/tBb781L5impVmmx2E9FRKF7d18XXzp4PGT2tdYMFY6wQqgxD0IwNZOiSJ0/K0fSi/OA== |
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.
node-abi error was encountered, so needed to bump this. electron/forge#667
@@ -181,6 +181,7 @@ export class DataContext { | |||
@cached | |||
get cloud () { | |||
return new CloudDataSource({ | |||
// @ts-ignore | |||
fetch: (...args) => this.util.fetch(...args), |
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.
To address error below. Open to better ideas here. https://app.circleci.com/pipelines/github/cypress-io/cypress/59948/workflows/efa9b4b3-12ca-4f83-a8d7-d5c9186f28d4/jobs/2493976
@@ -16,7 +16,7 @@ | |||
"types": [ | |||
"cypress", | |||
"./support", | |||
"./node_modules/@types/node" | |||
"node" |
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.
This comment was marked as resolved.
This comment was marked as resolved.
@MikeMcC399 Removed that reference. |
Additional details
Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?