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

USWDS - Update tool-versions file with current Node LTS version #5932

Merged
merged 1 commit into from
May 22, 2024

Conversation

amyleadem
Copy link
Contributor

@amyleadem amyleadem commented May 17, 2024

Summary

Updated the listed Node version in .tool-versions to the current LTS version, 20.13.1. Updating to this version from 20.9.0 resolves a MaxListenersExceeded warning during npm install.

Breaking change

This is not a breaking change.

Related issue

Closes #5927

Related pull requests

N/A - No changelog because it is not a user facing change

Preview link

N/A

Problem statement

USWDS currently specifies Node version 20.9.0 in its .tool-versions file. We encountered the following warnings during npm install when using node v20.9.0:

(node:18763) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [TLSSocket]. Use emitter.setMaxListeners() to increase limit

Updating to the current node LTS version (20.13.1) resolves this warning. We should update tool-versions to use this version of Node instead.

Testing and review

  1. Confirm that Node 20.13.1 is the current LTS version
  2. Confirm that updating to this version of Node resolves the memory leak warning

@amyleadem amyleadem changed the title USWDS - Update tool version USWDS - Update tool-version to Node LTS May 17, 2024
@amyleadem amyleadem marked this pull request as ready for review May 17, 2024 19:57
@amyleadem amyleadem changed the title USWDS - Update tool-version to Node LTS USWDS - Update tool-versions file with current Node LTS version May 17, 2024
@mejiaj
Copy link
Contributor

mejiaj commented May 17, 2024

I wasn't able to reproduce the original issue. No issues found with installing with node 20.13.1 though. I'm on an M1 mac.

@amyleadem amyleadem requested a review from thisisdano May 17, 2024 22:12
Copy link
Contributor

@mahoneycm mahoneycm left a comment

Choose a reason for hiding this comment

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

lgtm!

@thisisdano thisisdano merged commit d240ac8 into develop May 22, 2024
5 checks passed
@thisisdano thisisdano deleted the al-tool-versions-update branch May 22, 2024 21:48
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.

USWDS - Update node version in .tool-versions file
4 participants