-
Notifications
You must be signed in to change notification settings - Fork 201
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
chore(deps): bump rimraf from 2.6.2 to 5.0.10 #358
Conversation
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.
thanks!
@Matthew-Mallimo do you think you could take another look at this? It would fix a number of deprecation warnings that are currently being printed by the library
Happy Thanksgiving!
@code-forger My opinion is that this one is ready to be merged while that one is not. That PR does the same (and more) but I see no problems in taking this first since it removes a dependency with a known memory leak. |
Merge conflict fixed |
Description
Fixes #355
Motivation and Context
How Has This Been Tested?
Types of Changes
Checklist:
What is the Impact to Developers Using Jest-Image-Snapshot?