-
Notifications
You must be signed in to change notification settings - Fork 15
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 dependencies and tests #180
Conversation
Updated to the latest versions of the dependencies and also updated the tests to use a css property that generates some prefixes since flexbox no longer needs prefixes for the default browserslist config.
@@ -1,3 +1,4 @@ | |||
--reporter spec | |||
--compilers coffee:coffee-script/register |
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.
@@ -1,2 +1,2 @@ | |||
p | |||
display: flex |
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.
Since browserslist uses > 0.5%, last 2 versions, Firefox ESR, not dead
as its default I swapped flexbox out for a css rule that gets prefixes. https://github.com/browserslist/browserslist
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.
Amazing, thank you!
Nice 🎉 Since this have been merged 21 days ago, do you have any plan to release it on npm @jescalan? |
In postcss config, we gave the browsers list to autoprefixer plugin directly. But the `browsers` property is deprecated and will be removed in next major version (see https://github.com/postcss/autoprefixer/releases/tag/9.6.0). We also use autoprefixer through autoprefixer-stylus, but it uses a very old version of autoprefixer (7.1.0) that doesn't understand the `dead` query that our browserslist config uses. A PR fixing this have been merged and will hopefuly be released soon, see jescalan/autoprefixer-stylus#180. That's why I left the custom browserslist config for autoprefixer-stylus in `build:css`, `build:css:utils` and `build:css:kss` scripts for now. But we should upgrade this module and stop using a custom browserslist config for it as soon as possible.
In postcss config, we gave the browsers list to autoprefixer plugin directly. But the `browsers` property is deprecated and will be removed in next major version (see https://github.com/postcss/autoprefixer/releases/tag/9.6.0). We also use autoprefixer through autoprefixer-stylus, but it uses a very old version of autoprefixer (7.1.0) that doesn't understand the `dead` query that our browserslist config uses. A PR fixing this have been merged and will hopefuly be released soon, see jescalan/autoprefixer-stylus#180. That's why I left the custom browserslist config for autoprefixer-stylus in `build:css`, `build:css:utils` and `build:css:kss` scripts for now. But we should upgrade this module and stop using a custom browserslist config for it as soon as possible.
Hey @drazik - just released here https://github.com/jescalan/autoprefixer-stylus/releases/tag/v1.0.0 |
Thanks a lot! |
Updated to the latest versions of the dependencies and also updated the tests to
use a css property that generates some prefixes since flexbox no longer needs
prefixes when using the default browserslist config.
Fix for: #179