Skip to content

chore(package): update postcss v5.0.6...v6.0.23 (dependencies) #735

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

Closed
wants to merge 1 commit into from

Conversation

mikeumus
Copy link

What kind of change does this PR introduce?

bugfix

Summary

Simple fix for #691

@jsf-clabot
Copy link

jsf-clabot commented Jun 25, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Breaking change

@mikeumus
Copy link
Author

@evilebottnawi, what does that mean? Any changes from me? Would I have to get the tests passing for this new version of PostCSS?

@alexander-akait
Copy link
Member

@mikeumus css-loader works on any version node@js, but postcss only for 4 version, it is breaking change

@mikeumus
Copy link
Author

@evilebottnawi, I see. Anything I can do on my end?

@alexander-akait
Copy link
Member

@mikeumus it is very big work 😞 first we should create new loader for css modules and remove this from default css-loader, you can start this

@mikeumus
Copy link
Author

@evilebottnawi, I see, okay.

@michael-ciniawsky michael-ciniawsky changed the title postcss-modules-scope incompatible with PostCSS v5.x chore(package): update postcss v5.0.6...v6.0.23 (dependencies) Aug 7, 2018
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed by v1.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants