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

Add ability to pass parameters to coffee #36

Open
anikkar opened this issue Sep 13, 2012 · 6 comments
Open

Add ability to pass parameters to coffee #36

anikkar opened this issue Sep 13, 2012 · 6 comments

Comments

@anikkar
Copy link

anikkar commented Sep 13, 2012

Would be nice if we could pass some parameters to the coffee command such as -l for lint

@ghost ghost assigned arboleya Sep 13, 2012
@arboleya
Copy link
Owner

Well noticed.

@arboleya arboleya reopened this Sep 17, 2012
@arboleya
Copy link
Owner

@anikkar Hi, do you think any other param is useful to Toaster? I can't think of any except the -l.

Btw, I just found this coffeelint which seems very interesting:
http://www.coffeelint.org/

Have you used it already? Do you have some feedbacks about it?

@anikkar
Copy link
Author

anikkar commented Dec 31, 2012

I am using coffeelint actually, and its very helpful. I use it as a pre-compile step.

As for additional parameters, your right, -l is the only one that seems useful at the moment.

@arboleya
Copy link
Owner

I'm not a lint user but as I understand by now, when coding coffee the important is not the lint of your JS but of your COFFEE. Is that right?

Have you abandoned the coffee -l after starting using coffeelint? Or both plays together? (I didn't find)

I'm wondering if both are useful to implement, or in this case, just coffeelint.

@anikkar
Copy link
Author

anikkar commented Jan 2, 2013

Generally speaking, your right, you want to ensure that the coffee lint is correct, but the JS check is a sanity check to ensure that the coffeescript is correctly translated to JS.

@arboleya
Copy link
Owner

@anikkar Hi, I just want to tell you that I'm discontinuing Toaster in favor of it's new fork named Polvo.

More info here:

https://github.com/serpentem/coffee-toaster
https://github.com/serpentem/coffee-toaster/wiki

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

2 participants