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

feat: Upgrade to node 20 #197

Closed
wants to merge 1 commit into from

Conversation

nicholascapo
Copy link

  • Update to actions/checkout@v4
  • Update to actions/setup-node@v4
  • Set engines config in package.json
  • Use node-version-file option for actions/setup-node
  • Configure action to run using node20
  • Fixes: Node 16 actions are deprecated #196

* Update to actions/checkout@v4
* Update to actions/setup-node@v4
* Set engines config in package.json
* Use node-version-file option for actions/setup-node
* Configure action to run using node20
* Fixes: inception-health#196

Signed-off-by: Nicholas Capo <[email protected]>
@@ -71,5 +71,8 @@
"classNameTemplate": "{classname}",
"includeConsoleOutput": "true",
"reportTestSuiteErrors": "true"
},
"engines": {
"node": "^20"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to confirm that this works correctly in CI.
I expect to get a node version that is >=20, <21 but not exactly 20.0.0

@nicholascapo
Copy link
Author

Reviews please.
CC: @nikordaris @longility

Copy link
Contributor

@nikordaris nikordaris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nicholascapo
Copy link
Author

Is there something I can do to help move this forward? 😄

Copy link

@v1v v1v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the record, this is a breaking change, see hashicorp/vault-action#526.

If possible I'd say to release v2

@nicholascapo
Copy link
Author

For the record, this is a breaking change, see hashicorp/vault-action#526.

If possible I'd say to release v2

That makes sense.
v2 sounds good to me.

@nicholascapo
Copy link
Author

@nikordaris are you able to approve this so that (at least) the CI runs? 😄

@nicholascapo
Copy link
Author

👋 Any word here?
Anything I can do to help?

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.

Node 16 actions are deprecated
3 participants