Rule for banner/Ratliff style indentation

Environment: latest ESLint in Visual Studio Code.

I’ve read the indent rule’s guide.

I don’t understand whether it’s possible to setup a rule to allow me to use “banner” AKA “Ratliff” style indentation? That’s when the closing brace is indented to the same level as the code block.

// eg control block
if (foo) {
  doSomething();
  }

// eg array declaration
var foo = [
  bar,
  baz,
  qux
  ];

// same for arrays, expressions, etc...

Right now ESLint complains that Expected indentation of 0 spaces but found 2. (indent).

[jr-grenoble]

I just issued a pull request for this, hopefully this can serve as a basis for the new feature.

On Sat, Dec 2, 2017 at 08:48 grokky1 [e-mail] wrote:

@platinumazure https://github.com/platinumazure I have what @jr-grenoble https://github.com/jr-grenoble posted. I basically just made that one change recommended at the top of this thread, but it seems @jr-grenoble https://github.com/jr-grenoble improved on that, so I’d use his post as the starting point.

I really hope we can get this baked into the library.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/eslint/eslint/issues/9361#issuecomment-348675513, or mute the thread https://github.com/notifications/unsubscribe-auth/AXUjplZYAuaVTp9BhFdkphQrwFeB0ndGks5s8QDagaJpZM4Pl8YV .

–

https://icon.me/jeanrene?from=emailsignature&access=65b8b3e1677d51e77b65c69e8bd3fdc30ecbebb6 Jeanrene Bouvier Head of Facts Haven [e-mail] View my Icon https://icon.me/jeanrene?from=emailsignature&access=65b8b3e1677d51e77b65c69e8bd3fdc30ecbebb6

[jr-grenoble]

@platinumazure I just played with a local version so far and I didn’t write any corresponding tests beyond checking on some of my own code that it produced the right results. Here’s the diff:

@@ -600,6 +600,9 @@ module.exports = {
                                 pattern: ":exit$"
                             }
                         }
+                    },
+                    banner: {
+                        type: "boolean"
                     }
                 },
                 additionalProperties: false
@@ -638,6 +641,7 @@ module.exports = {
             ObjectExpression: 1,
             ImportDeclaration: 1,
             flatTernaryExpressions: false,
+            banner: false,
             ignoredNodes: []
         };
 
@@ -783,6 +787,7 @@ module.exports = {
          */
         function addElementListIndent(elements, startToken, endToken, offset) {
 
+
             /**
              * Gets the first token of a given element, including surrounding parentheses.
              * @param {ASTNode} element A node in the `elements` list
@@ -803,7 +808,7 @@ module.exports = {
                 startToken,
                 typeof offset === "number" ? offset : 1
             );
-            offsets.setDesiredOffset(endToken, startToken, 0);
+            offsets.setDesiredOffset(endToken, startToken, options.banner ? 1 : 0);
 
             // If the preference is "first" but there is no first element (e.g. sparse arrays w/ empty first slot), fall back to 1 level.
             if (offset === "first" && elements.length && !elements[0]) {
@@ -931,7 +936,7 @@ module.exports = {
                     });
                 }
 
-                offsets.setDesiredOffset(rightParen, leftParen, 0);
+                offsets.setDesiredOffset(rightParen, leftParen, options.banner ? 1 : 0);
             });
         }