Skip to content
This repository has been archived by the owner on Mar 18, 2019. It is now read-only.

Throw an error if cssnext is used with default options not as the first plugin #175

Closed
MoOx opened this issue Jul 1, 2015 · 4 comments · Fixed by #238
Closed

Throw an error if cssnext is used with default options not as the first plugin #175

MoOx opened this issue Jul 1, 2015 · 4 comments · Fixed by #238

Comments

@MoOx
Copy link
Owner

MoOx commented Jul 1, 2015

Some people include cssnext after other plugins, but since cssnext handle import by default, it should be the first so, if possible:

  • if cssnext has default options || import to true && import hasn't been piped yet, should throw if not the first plugin
  • if cssnext has default options || import to true && import has already been piped, should disable import (& should emit a warning ?) if not the first plugin
  • if cssnext has import to false, just check nothing, people probably know what they are doing

That would avoid people doing mistake.

@jednano
Copy link
Contributor

jednano commented Jul 27, 2015

I have nested and nested-props running on each import, which is why I also have nested and nested props running before cssnext. Basically, I don't want it trying to unwrap nested syntax more than once for the same code, so it unwraps nesting on the non-imported code first, then unwraps nesting on each import as they come.

@MoOx
Copy link
Owner Author

MoOx commented Jul 27, 2015

What you are asking will be doable with this option #149

@jednano
Copy link
Contributor

jednano commented Jul 27, 2015

Thanks! I'll follow that thread.

@MoOx MoOx mentioned this issue Dec 29, 2015
5 tasks
@MoOx MoOx closed this as completed in #238 Jan 4, 2016
@MoOx
Copy link
Owner Author

MoOx commented Jan 4, 2016

Useless now that (postcss-)cssnext doesn't handle import anymore.

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 a pull request may close this issue.

2 participants