-
Notifications
You must be signed in to change notification settings - Fork 41
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
Update 'browserlist', 'optipng-bin', and 'postcss-cli' to latest version #1173
Conversation
@@ -80,7 +80,7 @@ | |||
"babel-loader": "^7.1.5", | |||
"babel-preset-es2015": "^6.24.1", | |||
"babel-preset-react": "^6.24.1", | |||
"browserslist": "^3.2.6", | |||
"browserslist": "^4.7.3", |
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.
https://github.com/browserslist/browserslist/blob/master/CHANGELOG.md no breaking changes
@@ -64,7 +64,7 @@ | |||
"eslint-config-prettier": "^6.6.0", | |||
"eslint-plugin-prettier": "^3.1.1", | |||
"eslint-plugin-react": "^7.16.0", | |||
"optipng-bin": "^5.1.0", | |||
"optipng-bin": "^6.0.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.
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.
Looking at the main readme on https://github.com/imagemin/optipng-bin, we should swap this one out for https://github.com/imagemin/imagemin-optipng rather than updating the version number
package.json
Outdated
@@ -94,7 +94,7 @@ | |||
"moment": "^2.24.0", | |||
"node-sass": "^4.12.0", | |||
"npm-run-all": "^4.1.5", | |||
"postcss-cli-simple": "^1.0.3", | |||
"postcss-cli-simple": "^3.0.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.
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.
Hm, given this is getting less than 200 installs a week (https://www.npmjs.com/package/postcss-cli-simple) should we be using the official https://www.npmjs.com/package/postcss-cli instead?
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.
Sure. I thought about that as well but I couldn't decide if we needed the full package (vs the "simple" version). Will update the PR to replace postcass-cli-simple
with postcass-cli
.
@@ -64,7 +64,7 @@ | |||
"eslint-config-prettier": "^6.6.0", | |||
"eslint-plugin-prettier": "^3.1.1", | |||
"eslint-plugin-react": "^7.16.0", | |||
"optipng-bin": "^5.1.0", | |||
"optipng-bin": "^6.0.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.
Looking at the main readme on https://github.com/imagemin/optipng-bin, we should swap this one out for https://github.com/imagemin/imagemin-optipng rather than updating the version number
package.json
Outdated
@@ -94,7 +94,7 @@ | |||
"moment": "^2.24.0", | |||
"node-sass": "^4.12.0", | |||
"npm-run-all": "^4.1.5", | |||
"postcss-cli-simple": "^1.0.3", | |||
"postcss-cli-simple": "^3.0.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.
Hm, given this is getting less than 200 installs a week (https://www.npmjs.com/package/postcss-cli-simple) should we be using the official https://www.npmjs.com/package/postcss-cli instead?
@Pomax PR updated! |
@mmmavis do we want to swap optipng-bin for imagemin-optipng in this PR, too, or should that be a separate issue? |
Related to #1159