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

Imports option #57

Closed
blond opened this issue Mar 30, 2015 · 3 comments
Closed

Imports option #57

blond opened this issue Mar 30, 2015 · 3 comments
Milestone

Comments

@blond
Copy link
Member

blond commented Mar 30, 2015

imports: 'include'. By Default.

--include-css — Include regular CSS on @import. The default value is true.

@blond blond self-assigned this Mar 30, 2015
@blond blond added this to the 1.3.0 milestone Mar 30, 2015
@blond blond mentioned this issue Mar 30, 2015
6 tasks
@blond blond removed their assignment Apr 6, 2015
@blond blond modified the milestones: 1.3.0, 2.0.0 Jun 3, 2015
@levonet levonet added ready and removed backlog labels Jun 5, 2015
@blond blond changed the title Include option Imports option Jun 15, 2015
@tavriaforever
Copy link
Contributor

@blond I do not agree with default.
Now on first step we open only styl imports, and pass the opening imports another css module – postcss
I propose to do by default – false

@blond
Copy link
Member Author

blond commented Jul 1, 2015

@tavriaforever default value should be suitable for BEM projects.

If value will be false — everyone will have to keep the source files for dist.

Import: this issue about option for enb-stylus, not for stylus renderer.

@blond
Copy link
Member Author

blond commented Jul 15, 2015

Fixed in #61

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

No branches or pull requests

3 participants