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

Support for Stylus preprocessor #159

Closed
seniorpreacher opened this issue Jan 12, 2014 · 51 comments

Comments

@seniorpreacher
Copy link

commented Jan 12, 2014

I see, you have a great new repo and you support more-and-more features and this is great! I like it! ;)

I just ask here again, if you plan to support stylus in the future?

In the older repo 8 other people +1'd the feature: https://github.com/csscomb/csscomb/issues/146
:) I hope you have time and energy to create it, because it would be awesome! 👍

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1519052-support-for-stylus-preprocessor?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).
@tonyganch

This comment has been minimized.

Copy link
Member

commented Jan 13, 2014

Yes, we plan to support Stylus in the future.

@seniorpreacher

This comment has been minimized.

Copy link
Author

commented Jan 13, 2014

Awesome news! thanks! 👍

@tonyganch tonyganch added the gonzales label Feb 16, 2014

@havenchyk

This comment has been minimized.

Copy link

commented Apr 21, 2014

👍

@nathanredblur

This comment has been minimized.

Copy link

commented Apr 28, 2014

I want this too. 🤘

@mishanga

This comment has been minimized.

Copy link
Contributor

commented Jun 21, 2014

@A

This comment has been minimized.

Copy link

commented Jun 27, 2014

1364727664497

@banzalik

This comment has been minimized.

Copy link

commented Jul 10, 2014

+1

@techtheist

This comment has been minimized.

Copy link

commented Jul 10, 2014

:'O +1

@reklatsmasters

This comment has been minimized.

Copy link

commented Jul 10, 2014

+1

@defmthd

This comment has been minimized.

Copy link

commented Jul 10, 2014

+1

@tonyganch

This comment has been minimized.

Copy link
Member

commented Jul 10, 2014

I'd love to explain why Stylus is not supported yet.
In order to add Stylus to CSScomb we need to teach Gonzales PE to parse it first.
Adding the new syntax to the parser is tricky because:

  1. There are cases when you can't tell for sure whether a line of code is a declaration or a mixin.
  2. Someone needs to spend time writing code.
  3. Someone needs to maintain the code (forever).

You can't just write a parser once and forget about it. New features are constantly added and someone needs to follow news and update parser's code.

I'm not able to do this by myself. But we all are developers here. So if you really need Stylus, help by sending a PR. I'll be happy to accept it.
Thank you.

@artbelov

This comment has been minimized.

Copy link

commented Jul 10, 2014

👍

@kizu kizu referenced this issue Jul 16, 2014
@ptmt

This comment has been minimized.

Copy link

commented Aug 3, 2014

+1

4 similar comments
@tronin

This comment has been minimized.

Copy link

commented Aug 3, 2014

+1

@tjunghans

This comment has been minimized.

Copy link

commented Aug 6, 2014

👍

@megatolya

This comment has been minimized.

Copy link

commented Aug 7, 2014

+1

@tarmolov

This comment has been minimized.

Copy link

commented Aug 29, 2014

+1

@tarmolov

This comment has been minimized.

Copy link

commented Aug 29, 2014

Maybe we should create a bounty for the issue ^^

@tadatuta

This comment has been minimized.

Copy link

commented Aug 29, 2014

👍

3 similar comments
@slmgc

This comment has been minimized.

Copy link

commented Sep 11, 2014

+1

@dayAlone

This comment has been minimized.

Copy link

commented Sep 15, 2014

+1

@tavriaforever

This comment has been minimized.

Copy link

commented Sep 18, 2014

👍

@gela-d

This comment has been minimized.

Copy link

commented Sep 19, 2014

We are also need to
+1

@albychok

This comment has been minimized.

Copy link

commented Sep 23, 2014

+1

3 similar comments
@tagir-yusupov

This comment has been minimized.

Copy link

commented Oct 3, 2014

+1

@mishcherin

This comment has been minimized.

Copy link

commented Oct 13, 2014

+1

@NikolayFrantsev

This comment has been minimized.

Copy link

commented Oct 28, 2014

+1

@resure

This comment has been minimized.

Copy link

commented Nov 25, 2014

+1

1 similar comment
@dimovmo

This comment has been minimized.

Copy link

commented Dec 2, 2014

+1

@sipayRT

This comment has been minimized.

Copy link

commented Dec 18, 2014

👍

1 similar comment
@igormukhingmailcom

This comment has been minimized.

Copy link

commented Jan 27, 2015

+1

@jt3k

This comment has been minimized.

Copy link

commented Jan 30, 2015

+99999

@ishaba

This comment has been minimized.

Copy link

commented Feb 13, 2015

+1

2 similar comments
@warmrobot

This comment has been minimized.

Copy link

commented Feb 13, 2015

+1

@i4got10

This comment has been minimized.

Copy link

commented Feb 16, 2015

+1

@tenorok

This comment has been minimized.

Copy link
Contributor

commented Mar 14, 2015

1️⃣

@tenebricosa

This comment has been minimized.

Copy link

commented Mar 17, 2015

++1

@tonyganch tonyganch referenced this issue Mar 26, 2015
3 of 5 tasks complete
@a-x-

This comment has been minimized.

Copy link

commented Apr 9, 2015

👍

@a-x-

This comment has been minimized.

Copy link

commented Apr 9, 2015

@Thingon1

This comment has been minimized.

Copy link

commented Apr 9, 2015

@a-x- Thats pretty cool, it would be nice if someone could wrap this in Sublime so I don't have to run node or fire up the command line every time.

@a-x-

This comment has been minimized.

Copy link

commented Apr 9, 2015

in our workflow similar tasks added to pre-commit hook.

@diessica

This comment has been minimized.

Copy link

commented May 28, 2015

@a-x-'s tip is worth checking out! it doesn't sort automatically though. unfortunately it works more like csslint rather than csscomb. :\

@denysdovhan

This comment has been minimized.

Copy link

commented Jun 17, 2015

I'm waiting too

@prostoVladislav

This comment has been minimized.

Copy link

commented Jun 23, 2015

+1

2 similar comments
@apostololeg

This comment has been minimized.

Copy link

commented Jun 29, 2015

+1

@jaybekster

This comment has been minimized.

Copy link

commented Jul 2, 2015

+1

@csscomb csscomb locked and limited conversation to collaborators Jul 2, 2015

@tonyganch

This comment has been minimized.

Copy link
Member

commented Jul 2, 2015

Please see my comment above: #159 (comment)

@tonyganch tonyganch removed the preprocessors label Aug 3, 2016

@obenjiro obenjiro added this to the 5.0.0 milestone Aug 9, 2017

@jdalton

This comment has been minimized.

Copy link
Collaborator

commented May 6, 2019

Please direct stylus support PRs to gonzales-pe.

@jdalton jdalton closed this May 6, 2019

@jdalton jdalton added status: wontfix and removed help wanted labels May 6, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
You can’t perform that action at this time.