-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Svgo v2 #329
Svgo v2 #329
Conversation
9cc956e
to
3802f2c
Compare
3802f2c
to
5606312
Compare
This pull request introduces 1 alert when merging 3808f70 into 32c796d - view on LGTM.com new alerts:
|
packages/react-svg-core/src/index.ts
Outdated
// SVGO Optimize | ||
export function optimize(opts: any = {}): (content: string) => Promise<string> { | ||
export function optimize(opts: any = {}) { | ||
opts = validateAndFix(opts); |
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.
Config format is different now. So validateAndFix might need to be changed as well.
3808f70
to
8e63536
Compare
This pull request introduces 1 alert when merging 8e63536 into 32c796d - view on LGTM.com new alerts:
|
8e63536
to
e06f063
Compare
@boopathi can you review this, please? We use your loader and it fails vulnerability check every build. |
Ref: #307
NodeJS releases status and END-OF-LIFE
error [email protected]: The engine "node" is incompatible with this module. Expected version ">=10.13.0". Got "8.17.0"
https://github.com/svg/svgo/blob/b37d90e12a87312bba87a6c52780884e6e595e23/package.json#L52-L54
Travis build
https://app.travis-ci.com/github/boopathi/react-svg-loader/jobs/550900558