Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

[WIP] Use postcss-cssnext #220

Closed
wants to merge 6 commits into from
Closed

Conversation

jednano
Copy link

@jednano jednano commented Sep 18, 2015

Based on #219.

This is just a work in progress for now.

@jednano jednano force-pushed the use-postcss-cssnext branch 5 times, most recently from ef0da09 to 97ea344 Compare September 18, 2015 18:59
@jednano jednano force-pushed the use-postcss-cssnext branch from 97ea344 to 1366c22 Compare September 18, 2015 19:07
@jednano
Copy link
Author

jednano commented Sep 18, 2015

@MoOx it looks like the async issue is the only thing failing the build at this time. Also, maybe we should move the features benchmarks to postcss-cssnext? What do you think?

@jednano jednano changed the title [WIP] Use postcss cssnext [WIP] Use postcss-cssnext Sep 18, 2015
@MoOx
Copy link
Owner

MoOx commented Sep 21, 2015

The async issue is a big deal because all existing runners are using it, so when we will change the API by removing the simple cssnext(string, options) in favor of the postcss style cssnext(options).process(string), we will loose compat with all runners. That why I planned to deprecate them (see #208 (comment)) first (and communicate about it) before upgrading cssnext itself (or not? to deprecate it directly?).

@jednano
Copy link
Author

jednano commented Sep 21, 2015

Totally makes sense.

@jednano jednano closed this Sep 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants