-
Notifications
You must be signed in to change notification settings - Fork 12
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-Compile - Dependencies: Use Node LTS 20 #78
Conversation
@@ -0,0 +1 @@ | |||
nodejs 20.9.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a .tool-versions
file for folks who use other version managers like asdf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Here is what I tested:
- Successfully ran the
uswds-sandbox
tests described in the PR description:- Ran
npm install
,npm run uswds:copyAssets
,npm run watch
without error - Confirmed that styles render as expected
- Ran
- Checked that references to previous node versions in docs have been updated to v20
- Confirmed README formatting hasn’t changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
- Installation and tasks run without errorr
- Testing on sandbox shows no regressions
- Read me page looks good
Use latest Node LTS 20 and update dependencies. Using latest LTS version of node and updated version of USWDS and SASS. Closes #71.
Dependencies
How to test
test-uswds-compile-78-node-update
branch.npm install
.npm run uswds:copyAssets
npm run watch