brace-style ignores braces around class body

Not very sure about this. I’ll post it here anyway so someone can have a look. Feel free to close if I’m horribly wrong.

Using this configuration:

{ 'brace-style': ['error', 'allman'] }

What did you do? Please include the actual source code causing the issue.

class Foo {
}

What did you expect to happen?

According to Allman style rules, the opening brace should appear on a new line, and so the code above should trigger an error.

What actually happened? Please include the actual, raw output from ESLint.

No error.

[platinumazure]

Rule enhancement Bugfix proposal

What rule do you want to change? brace-style

Does this change cause the rule to produce more or fewer warnings? More

How will the change be implemented? (New option, new default behavior, etc.)? New default behavior, ideally (this is a missed use case)

Please provide some example code that this change will affect:

/* eslint-env es6 */
/* eslint brace-style: ["error", "allman"] */

class Foo {
}

What does the rule currently do for this code? No error

What will the rule do after it’s changed? Warn for incorrect brace style

Note: This applies to all brace styles, not just Allman.

I’ll champion.

[fasttime]

@platinumazure I have zero experience working on the ESLint codebase, just some familiarity with parser logic and ASTs. I’ll leave this one to @not-an-aardvark, but I’d be glad to help out another time.