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

Updating project to node 18 #965

Merged
merged 2 commits into from
Apr 24, 2023
Merged

Updating project to node 18 #965

merged 2 commits into from
Apr 24, 2023

Conversation

kriswest
Copy link
Contributor

@kriswest kriswest commented Apr 21, 2023

resolves #964

Updating lock files, github workflows and adding an .nvmrc file (for netlify) and .node-version (other tools) for node 18 in both the root of the repository and the website directory (where netlify will read from).
Removing engines from package.json as it ends up in node module and should not.
Rolling npm module version so that it releases (without engines restriction).

Correct version is selected by netlify:
image

Previews all seem good:

Updating lock files, github workflows and adding an .nvmrc file (for netlify) for node 18.
Removing engines from package.json as it ends up in node module and should not.
Rolling npm module version so that it releases (without engines restriction)
@kriswest kriswest added website project infrastructure toolbox Apps or code that isn't part of the standard but supports it use labels Apr 21, 2023
@kriswest kriswest requested a review from a team April 21, 2023 16:12
@netlify
Copy link

netlify bot commented Apr 21, 2023

Deploy Preview for fdc3 ready!

Name Link
🔨 Latest commit 4b50d2b
🔍 Latest deploy log https://app.netlify.com/sites/fdc3/deploys/6442b6bb408bfb00086f8b2c
😎 Deploy Preview https://deploy-preview-965--fdc3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

…tools to .nvmrc) and replicating in website directory
@kriswest kriswest requested a review from robmoffat April 21, 2023 16:19
@kriswest kriswest mentioned this pull request Apr 21, 2023
@kriswest kriswest merged commit cb32e60 into master Apr 24, 2023
Copy link
Contributor

@hughtroeger hughtroeger left a comment

Choose a reason for hiding this comment

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

👍

@kriswest kriswest deleted the update-to-node-18 branch April 25, 2023 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project infrastructure toolbox Apps or code that isn't part of the standard but supports it use website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update all repo projects to support node 18
3 participants