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

deps: use node lts #7178

Merged
merged 4 commits into from
Sep 11, 2023
Merged

deps: use node lts #7178

merged 4 commits into from
Sep 11, 2023

Conversation

tynes
Copy link
Contributor

@tynes tynes commented Sep 11, 2023

Description

Upgrade from node.js v16 to v18. v16 starts end of life on 9/11/2023, meaning it will no longer get security updates. v18 is currently LTS, https://github.com/nodejs/release#release-schedule for additional information.

We previously were not able to upgrade node.js versions because we were using an older version of hardhat that did not support newer versions of node.js but that is no longer the case.

Upgrade from node.js v16 to v18. v16 starts end of life on
9/11/2023, meaning it will no longer get security updates.
v18 is currently LTS, https://github.com/nodejs/release#release-schedule
for additional information.

We previously were not able to upgrade node.js versions because
we were using an older version of hardhat that did not support newer
versions of node.js but that is no longer the case.
@tynes tynes requested a review from a team as a code owner September 11, 2023 11:19
@tynes tynes requested a review from Inphi September 11, 2023 11:19
@mergify mergify bot added the A-ops Area: ops label Sep 11, 2023
.nvmrc Show resolved Hide resolved
ops/docker/Dockerfile.packages Outdated Show resolved Hide resolved
Co-authored-by: Will Cory <willcory10@gmail.com>
@mergify
Copy link
Contributor

mergify bot commented Sep 11, 2023

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Sep 11, 2023

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot added the S-on-merge-train Status: This PR is in the merge queue label Sep 11, 2023
@mergify
Copy link
Contributor

mergify bot commented Sep 11, 2023

This PR is next in line to be merged, and will be merged as soon as checks pass.

@OptimismBot OptimismBot merged commit bc602c0 into develop Sep 11, 2023
@OptimismBot OptimismBot deleted the deps/node-lts branch September 11, 2023 16:38
@mergify mergify bot removed the S-on-merge-train Status: This PR is in the merge queue label Sep 11, 2023
@github-actions github-actions bot mentioned this pull request Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ops Area: ops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants