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
fix(deps) : update arborist package to get rid of deprecated warning #3559
Conversation
0955822
to
da5dcb2
Compare
@JamesHenry @fahslaj eslint is complaining. Can I add the failure to the esignore patterns |
@amorscher The lint error is occurring within a file nested within |
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.
See comment
d285f21
to
765463f
Compare
@JamesHenry so I also updated dependant packages of arborist so that all modules are hoisted. |
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.
Thank you @amorscher!
Partially fixes #3550
Description
Updates to the actual version of npmcli/arborist which causes following deprecation warning to disappear.
npm WARN deprecated readdir-scoped-modules@1.1.0: This functionality has been moved to @npmcli/fs
Motivation and Context
Partially fixes #3550
How Has This Been Tested?
Existing unit, integration and e2e tests.
Types of changes
Checklist: