-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
ci: enable arm64 macOS tests #24832
ci: enable arm64 macOS tests #24832
Conversation
fb9983a
to
4712a86
Compare
0fbb5d8
to
2c21880
Compare
947ad8f
to
a470fa8
Compare
a470fa8
to
0164ca9
Compare
7a04de7
to
c9af89c
Compare
77a5c96
to
f224cc1
Compare
02ff21f
to
00f2f92
Compare
150c344
to
48903f6
Compare
48903f6
to
8e8ad0b
Compare
908d9cf
to
2556791
Compare
5914edc
to
12510c6
Compare
ab4ea78
to
d3ac340
Compare
4d3e40a
to
8b37da5
Compare
Merging as CI failure is unrelated to PR changes. |
No Release Notes |
@@ -17,6 +17,7 @@ steps: | |||
node script/download-circleci-artifacts.js --buildNum=$CIRCLE_BUILD_NUM --name=dist.zip --dest=$ZIP_DEST | |||
cd $ZIP_DEST | |||
unzip -o dist.zip | |||
xattr -cr Electron.app |
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.
@jkleinsc What is the reasoning behind clearing the extended attributes here?
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.
@jviotti just to make sure the quarantine attribute is removed from the downloaded file.
Description of Change
This PR enables testing of arm64 macOS builds.
Checklist
npm test
passesRelease Notes
Notes: none