Skip to content

Raise Node version to 20 in package.json and GitHub Actions.#1448

Merged
demiankatz merged 2 commits into
UniversalViewer:devfrom
FalveyLibraryTechnology:node20
Jun 13, 2025
Merged

Raise Node version to 20 in package.json and GitHub Actions.#1448
demiankatz merged 2 commits into
UniversalViewer:devfrom
FalveyLibraryTechnology:node20

Conversation

@demiankatz

@demiankatz demiankatz commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

Node 18 has reached end of life, so it's time to move on to Node 20.

Resolves #1433.

@vercel

vercel Bot commented Jun 12, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
universalviewer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 13, 2025 11:27am

LanieOkorodudu
LanieOkorodudu previously approved these changes Jun 13, 2025

@LanieOkorodudu LanieOkorodudu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @demiankatz for taking the time to upgrade to Node 20! This should help with performance and keep us aligned with long-term support. I believe it's good to merge since all tests have passed.

@demiankatz

Copy link
Copy Markdown
Contributor Author

Thanks, @LanieOkorodudu! I just found another config file with an outdated node version in it, so I've adjusted this PR to update that as well. I'm not sure if or how it's currently used, but it seemed wise to be consistent.

@LlGC-jop LlGC-jop left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks fine to me

@demiankatz demiankatz merged commit 4291bce into UniversalViewer:dev Jun 13, 2025
4 checks passed
@demiankatz demiankatz deleted the node20 branch June 13, 2025 13:05
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.

Raise minimum Node version to 20

3 participants