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

log the rule with information about error #219

Closed
iamstarkov opened this issue Jan 28, 2014 · 6 comments
Closed

log the rule with information about error #219

iamstarkov opened this issue Jan 28, 2014 · 6 comments

Comments

@iamstarkov
Copy link
Contributor

it would be hepfull with configuring

@markelog
Copy link
Member

"logout the rule with information about error" what do mean by that?

@iamstarkov
Copy link
Contributor Author

I mean that node-jscs should log not only error, but error with rule responsible rule

@pickhardt
Copy link

Even better: use error codes, like Closure. https://code.google.com/p/closure-linter/source/browse/trunk/closure_linter/errors.py

Then have a command to map from error codes to responsible rule(s).

Example:

"jscs explain E101"

@pickhardt
Copy link

One reason this is better is that you can do things with specific error codes. For instance, if E101 = Line too long, then you could have a comment to allow a really long regex line:

var x = /myreallyreallylongregexgoeshere/; // jscs ignore E101

@iamstarkov
Copy link
Contributor Author

Yes, adding rule code is good idea

@markelog
Copy link
Member

markelog commented Feb 1, 2014

Okay let's focus on #224 then

@markelog markelog closed this as completed Feb 1, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants