-
Notifications
You must be signed in to change notification settings - Fork 55
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(ci): add arm64 for macOS #1212
base: master
Are you sure you want to change the base?
Conversation
3119e22
to
b525cd3
Compare
Co-authored-by: Ludovic Chenut <ludovic@status.im>
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.
LGTM
@@ -75,7 +81,7 @@ jobs: | |||
- name: Setup Go | |||
uses: actions/setup-go@v5 | |||
with: | |||
go-version: '~1.15.5' | |||
go-version: '~1.16.0' # That's the minimum Go version that works with arm. |
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.
Would you mind updating the README accordingly so this gets reflected? That is, that we support 1.16.0
onwards.
@@ -87,8 +93,8 @@ jobs: | |||
with: | |||
path: nimbledeps | |||
# Using nim.branch as a simple way to differentiate between nimble using the "pkgs" or "pkgs2" directories. | |||
# The change happened on Nimble v0.14.0. | |||
key: nimbledeps-${{ matrix.nim.branch }}-${{ hashFiles('.pinned') }} # hashFiles returns a different value on windows | |||
# The change happened on Nimble v0.14.0. Also forcing the deps to be reinstalled on each os and cpu. |
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.
What changed for this to matter?
This PR adds the macOS 14 GitHub runner that uses the arm64 cpu.