Skip to content
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): upgrade to broccoli-postcss 4 #124

Merged
merged 1 commit into from
Nov 6, 2018

Conversation

alexlafroscia
Copy link
Contributor

@alexlafroscia alexlafroscia commented Nov 2, 2018

We ran into a bug due to the previous broccoli-postcss's strict reliance on a specific broccoli-persistent-filter version

The broccoli-postcss upgrade brings the PostCSS version up to 7. This might mean that landing this PR would require a breaking change here, too.

Both the major-version bump in PostCSS and the broccoli plugin drop Node 4 and 9 support, but this addon never tried to be compatible with either of those anyway, based on the declared engine. PostCSS does have some other breaking changes around browser support, however, but it's mostly for dead browsers. You can find the relevant changelog here.

@dfreeman
Copy link
Member

dfreeman commented Nov 6, 2018

Thanks @alexlafroscia!

We dropped Node 4 and 9 support in the 1.0 release, so we should be good there. Based on the changelog, it doesn't look like this should be breaking for our consumers, so I'm happy to put this out in a patch release.

@dfreeman dfreeman merged commit 2fa7cc4 into salsify:master Nov 6, 2018
@dfreeman
Copy link
Member

dfreeman commented Nov 6, 2018

Released in ember-css-modules@1.0.2

@alexlafroscia
Copy link
Contributor Author

Awesome, thanks so much @dfreeman !

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

Successfully merging this pull request may close these issues.

2 participants