-
Notifications
You must be signed in to change notification settings - Fork 0
fix(deps): update dependency chalk to v6 #22
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,7 +30,7 @@ | |
| "author": "Sulthon", | ||
| "license": "MIT", | ||
| "dependencies": { | ||
| "chalk": "^5.6.2", | ||
| "chalk": "^6.0.0", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] correctness π‘ Verify that the target runtime environment supports Node.js 22. If support for older Node versions is required, pin chalk to ^5.6.2. If Node 22 is guaranteed, update the 'engines' field in package.json to '>=22' to enforce this requirement explicitly. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] correctness π‘ Downgrade chalk to ^5.x to maintain compatibility with Node 16+, or explicitly update the 'engines' field in package.json to '>=22' if this environment constraint is intentional. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] correctness π‘ Revert to 'chalk' ^5.6.2 for broader compatibility, or explicitly update the 'engines' field in package.json to 'node': '>=22' to document the new requirement. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] correctness π‘ If backward compatibility with Node 18 or 20 is required, remain on chalk 5.x. Otherwise, update the root 'engines' field in package.json to '{ "node": ">=22" }' to explicitly declare the new requirement. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] correctness π‘ Ensure the project's 'engines' field in package.json is updated to '>=22', or downgrade 'chalk' to ^5.6.2 to maintain compatibility with Node 16, 18, and 20. |
||
| "commander": "^15.0.0", | ||
| "simple-git": "^3.20.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.
[HIGH] correctness
Dependency 'chalk' was upgraded to v6.0.0, which requires Node.js >=22, but the project engine requirement (seen in package-lock.json) is '>=20.0.0'. This will cause installation or runtime failures for users on Node 20 or 21.
π‘ Update the 'engines.node' field in package.json to '>=22', or downgrade 'chalk' to v5.x to maintain compatibility with Node 20.