M1 is now available in GitHub CI, but we don't currently have a plan to use it, so remove the comment.
ci: remove note re M1 usage #28823
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:ci_remove_m1_note changing 1 files +1 −1-
fanquake commented at 3:11 PM on November 8, 2023: member
-
8cbb619691
ci: remove note re M1 usage
M1 is now available in GitHub CI, but we don't currently have a plan to use it, so remove the comment.
-
DrahtBot commented at 3:11 PM on November 8, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Tests on Nov 8, 2023
-
maflcko commented at 3:40 PM on November 8, 2023: member
lgtm ACK 8cbb6196913b22006dac75f719a2834ab0d6c94f
- hebasto approved
-
hebasto commented at 5:37 PM on November 8, 2023: member
M1 is now available in GitHub CI, but we don't currently have a plan to use it...
... for the only reason that it is not free for now. I believe, we still have a plan to use a free M1 runner in the future.
... so remove the comment.
Yes. It is outdated and misleading with the current wording.
ACK 8cbb6196913b22006dac75f719a2834ab0d6c94f.
-
achow101 commented at 6:20 PM on November 8, 2023: member
ACK 8cbb6196913b22006dac75f719a2834ab0d6c94f
- achow101 merged this on Nov 8, 2023
- achow101 closed this on Nov 8, 2023
- fanquake deleted the branch on Nov 9, 2023
- bitcoin locked this on Nov 8, 2024
