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 action to use Node 16 #122

Merged
merged 2 commits into from
Dec 9, 2021
Merged

Update action to use Node 16 #122

merged 2 commits into from
Dec 9, 2021

Conversation

ja-bravo
Copy link
Contributor

@ja-bravo ja-bravo commented Dec 9, 2021

Github actions now support Node v16 as it has entered Active LTS.

Ref: actions/runner#1439

@ja-bravo ja-bravo requested review from simoneb and Eomm December 9, 2021 13:15
@ja-bravo ja-bravo merged commit dd019b7 into main Dec 9, 2021
@ja-bravo ja-bravo deleted the update-to-node16 branch December 9, 2021 14:34
@simoneb
Copy link
Contributor

simoneb commented Dec 9, 2021

folks let's not rush merges please. if you're not sure, wait for my review.

this was missed https://github.com/nearform/github-action-notify-release/blob/main/.github/workflows/ci.yml#L10
plus I would have expected a change in package-lock too, which is not there.

plus, the setup-node action can read the version from .nvmrc. in other words, this change is missing quite a few things

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.

None yet

3 participants