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 chalk from ^1.1.3
to ^4.0.0
#310
chore(deps): bump chalk from ^1.1.3
to ^4.0.0
#310
Conversation
Bumps [chalk][1] from `^1.1.3` to `^4.0.0`. - [Release notes](https://github.com/chalk/chalk/releases) - [v2.0.0](https://github.com/chalk/chalk/releases/tag/v2.0.0) - [v3.0.0](https://github.com/chalk/chalk/releases/tag/v3.0.0) - [v4.0.0](https://github.com/chalk/chalk/releases/tag/v4.0.0) - [Commits](chalk/chalk@v1.1.3...v4.0.0) Chalk's major changes were: - Minimum required Node version is Node v10 (v4) - `chalk.constructor` has been replaced with `chalk.Instance` (v3) - `chalk.enabled` has been replaced with `chalk.level` (v3) Jest v28 uses Chalk v4 as a dependency, so using the same version in jest-image-snapshot will mean users only need to download one version of chalk. [1]: https://www.npmjs.com/package/chalk
"chalk": { | ||
"version": "1.1.3", | ||
"resolved": "https://registry.npmjs.org/chalk/-/chalk-1.1.3.tgz", | ||
"integrity": "sha512-U3lRVLMSlsCfjqYPbLyVv11M9CPW4I728d6TCKMAOJueEeB9/8o+eSsMnxPJD+Q+K909sdESg7C+tIkoH6on1A==", | ||
"dev": true, |
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.
chalk
v1.1.3
is now only required as a "dev"
dependency, and should completely disappear from your package-lock.json
file once you update eslint-config-amex
to a newer version.
const ChalkMock = () => ({ | ||
bold: { | ||
red: input => input, | ||
}, | ||
}); |
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.
This doesn't seem to do anything, so I've removed it.
I'm guessing that maybe this code was used in earlier tests but is no longer needed.
🎉 This PR is included in version 6.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Bumps chalk from
^1.1.3
to^4.0.0
.Chalk's major changes were:
chalk.constructor
has been replaced withchalk.Instance
(v3)chalk.enabled
has been replaced withchalk.level
(v3)Motivation and Context
Jest v28 uses Chalk v4 as a dependency, so using the same version in jest-image-snapshot will mean users only need to download one version of chalk.
How Has This Been Tested?
jest
tests pass.I've tried testing this branch on my own project (remark-mermaid-dataurl) using the following, and it works fine.
Additionally, my
package-lock.json
file lost 107 lines, so a bunch of packages were no longer needed.Types of Changes
Checklist:
[ ] My change requires a change to the documentation and I have updated the documentation accordingly.No change required.What is the Impact to Developers Using Jest-Image-Snapshot?
No major changes, except that their
package-lock.json
file should be slightly simpler, and theirnode_modules
folder slightly smaller (assuming that they have no other packages that rely onchalk
v1).