Skip to content

feat: build Cypress for linux-arm64#22252

Merged
flotwig merged 90 commits into
developfrom
linux-arm64
Jun 15, 2022
Merged

feat: build Cypress for linux-arm64#22252
flotwig merged 90 commits into
developfrom
linux-arm64

Conversation

@flotwig
Copy link
Copy Markdown
Contributor

@flotwig flotwig commented Jun 10, 2022

User facing changelog

  • Cypress is now available natively on arm64/aarch64 systems running Linux.
  • Upgraded Electron from 18.0.4 to 18.3.0.

Additional details

To do

  • Filter out from PR builds

Steps to test

  • Use the latest pre-release comment on this PR and attempt to install via npm i inside of a Docker for M1 container.

How has the user experience changed?

PR Tasks

Comment thread package.json
"del": "3.0.0",
"detect-port": "^1.3.0",
"electron": "18.0.4",
"electron": "18.3.0",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

A regression was introduced in Electron 18 that broke compatibility with Linux Arm, but it's fixed.

@flotwig flotwig marked this pull request as ready for review June 11, 2022 03:47
@flotwig flotwig requested a review from a team as a code owner June 11, 2022 03:47
@flotwig flotwig requested review from jennifer-shehane and removed request for a team June 11, 2022 03:47
@tbiethman tbiethman requested review from rockhold and ryanthemanuel and removed request for jennifer-shehane June 13, 2022 15:39
Comment thread cli/.eslintrc.json Outdated
"message": "Do not use `arch()` to detect the user's machine architecture. Use utils.getRealArch() instead."
},
{
"selector": "CallExpression[callee.object.name='process'][callee.property.name='arch']",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

also need os.arch

Base automatically changed from use-m1-runners to develop June 15, 2022 15:24
@borecz
Copy link
Copy Markdown

borecz commented Jul 7, 2022

Is this now included in the v10.3.0?

@flotwig
Copy link
Copy Markdown
Contributor Author

flotwig commented Jul 11, 2022

@borecz Yes, this PR is in Cypress 10.2.0 and above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide binary for arm64 architecture (Linux)

4 participants