Skip to content
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

Update Node.js from v12 to v16 for Apple M1 support #59

Merged
merged 4 commits into from
Jul 4, 2022

Conversation

hannesa2
Copy link
Contributor

@hannesa2 hannesa2 commented Jul 2, 2022

According to the release policy of Node.js,
Node.js v12 reached EOL (end-of-life).

https://nodejs.org/en/about/releases/

I think ruby/setup-ruby is ready to update Node.js to v16
because GitHub Actions runners support v16.

actions/runner#772

An I hope this action runs now on a Apple Silicon M1.
close #57

@hannesa2 hannesa2 changed the title Update Node.js from v12 to v16 Update Node.js from v12 to v16 for Apple M1 support Jul 2, 2022
According to the release policy of Node.js,
Node.js v12 reached EOL (end-of-life).

https://nodejs.org/en/about/releases/

I think ruby/setup-ruby is ready to update Node.js to v16
because GitHub Actions runners support v16.

actions/runner#772
@hannesa2
Copy link
Contributor Author

hannesa2 commented Jul 3, 2022

@Malinskiy any chance to get this merged to make action work on Apple Silicon M1 ?
The issue you see here e.g https://github.com/hannesa2/CVScanner/runs/7162165835?check_suite_focus=true

@Malinskiy
Copy link
Owner

Sure thing, but we need to fix https://github.com/Malinskiy/action-android/blob/2e4591be80bdbe897c601bd89046582f28e7d603/prepare-for-release.sh also to use a newer docker image with node 16

Might also need to update the package.json files as well?

@hannesa2
Copy link
Contributor Author

hannesa2 commented Jul 3, 2022

Ok, I did prepare-for-release.sh
but I've no clue what to do with package.json

@hannesa2
Copy link
Contributor Author

hannesa2 commented Jul 3, 2022

I use now node16 (as good as I can) everywhere

@Malinskiy Malinskiy merged commit a8b3137 into Malinskiy:master Jul 4, 2022
@hannesa2 hannesa2 deleted the Update.Node.js.v16 branch July 4, 2022 05:04
@hannesa2 hannesa2 mentioned this pull request Jul 4, 2022
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.

using in a custom Apple M1 runner / node12 node16
2 participants