-
Notifications
You must be signed in to change notification settings - Fork 230
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
Allow M1 building & testing via arm64 JDK #4898
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4898 +/- ##
=======================================
Coverage 38.46% 38.46%
=======================================
Files 166 166
Lines 12353 12353
=======================================
Hits 4751 4751
Misses 7602 7602 Continue to review full report at Codecov.
|
74460d7
to
43bd433
Compare
43bd433
to
83cfef1
Compare
83cfef1
to
2717822
Compare
@skhamis Is this ready to land? |
Hey Ryan, I haven't been able to run the full test just yet (I need to make sure arm64 jdk compiles & tests all the way up to fenix) but I should be able to soon! |
2717822
to
989caec
Compare
Sorry, I don't think I'm in a position to meaningfully review this. |
989caec
to
c7f25ec
Compare
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.
@skhamis I'm giving this a rubber stamp approval because I can't test this (yet). Sorry for the delay. I think I saw that you closed this a while back and dismissed it. 🫣
c7f25ec
to
b39ccdc
Compare
This reverts commit 15b00bf.
update cross compile tool for M1 macs
update cross compile tool for M1 macs
update cross compile tool for M1 macs
update cross compile tool for M1 macs
update cross compile tool for M1 macs
update cross compile tool for M1 macs
update cross compile tool for M1 macs
Pull Request checklist
[ci full]
to the PR title.