Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Rule change request: indent: allow "off" for "outerIIFEBody"

See original GitHub issue

What rule do you want to change? indent

Does this change cause the rule to produce more or fewer warnings? Fewer warnings when applied (but the default behavior would be the same)

How will the change be implemented? (New option, new default behavior, etc.)? New option, default behavior unchanged

Several options for the indent rule accept the string "off" to disable the check and let the user do it however they want. But a few of them do not, such as "outerIIFEBody". I would like to be able to set "outerIIFEBody" to "off". Currently I have to choose between 0 or 1 but both choices enforce something, and I don’t want to enforce anything.

Also, I find it curious that several of the options support the "off" setting but only "outerIIFEBody" and a few others don’t.

Please provide some example code that this change will affect:

// File 1
(function() {
    a();
)();

// File 2
(function() {
b();
)();

What does the rule currently do for this code? One of the files will cause a warning, depending whether I choose 0 or 1 for the "outerIIFEBody" rule.

What will the rule do after it’s changed? With the "off" option, eslint would not give a warning for any of those cases, while still checking the other indentation rules for the other options.

Are you willing to submit a pull request to implement this change? Sadly I don’t have time for this at the moment…