ability to ignore specific rule on specific line

[mikesherov]

e.g.

// jscs maximumLineLength: false
kjkjfsdnfkndsknjjkdfsnkjsfdknfjdsknjfskjsndfkjnfskjnfdsknjfkjnsdkjnfkjnsdfkjnfskjnskdjn
// jscs maximumLineLength: true

Note, however, this does not imply inline re-configuration of rules. Only the ability to ignore or unignore an already configured rule for the line.

[qfox]

What if it was disabled by config?

// jscs maximumLineLength: false
kjkjfsdnfkndsknjjkdfsnkjsfdknfjdsknjfskjsndfkjnfskjnfdsknjfkjnsdkjnfkjnsdfkjnfskjnskdjn
// jscs maximumLineLength: null

Like that?

[mikesherov]

@zxqfox sorry, I'm not sure I understand what you're asking. This will allow the user to disable or enable reporting of errors for a specific rule. It will not allow re-configuration of a rule inline. However, #487 will allow you to specify the config for an individual file at the top of that file.

[qfox]

@mikesherov Seems like it enable or disable "global" configuration for the rest of file. But how to disable this?

For example. I have global config:

{
  "maximumLineLengh": 50
}

And a file with inline config:

//// here maximumLineLength is 50
// jscs maximumLineLength: 80
//// here maximumLineLength is 80
kjkjfsdnfkndsknjjkdfsnkjsfdknfjdsknjfskjsndfkjnfskjnfdsknjfkjnsdkjnfkjnsdfkjnfskjnskdjn
// jscs maximumLineLength: false
//// here maximumLineLength is disabled, but how to rollback this to 50?

[mikesherov]

The rules go like this:

Configure using .jscsrc or config at top of file, enable or disable configured rule inline.

There is no situation where you can do maximumLineLength: 80 inline.

[qfox]

Ah. Got it. It's fine. Not so flexible as it could be but still fine.

[qfox]

@mikesherov For more sense it could be like enable/disable instead of true/false.

// jscs maximumLineLength: disable

[mrjoelkemp]

I'd like to tackle this, unless @markelog wants it ;), to get the 1.6 milestone moving. It's not too far away from what I did for the file-wide config. Have we settled on the api?

[mikesherov]

I'm almost done. It'll be done today. Thanks for the offer though @mrjoelkemp