Enable rule `padded-blocks` about eslint-config-stylelint HOT 10 CLOSED

The block-spacing rule is already enabled:

• eslint-config-stylelint/blob/master/eslintrc.json#L14 "block-spacing": 2,

from eslint-config-stylelint.

@ntwb sorry, i mean padded-blocks rule

from eslint-config-stylelint.

Ref http://eslint.org/docs/rules/padded-blocks

As the rule supports --fix here's a gist of the ~21k lines diff after manually adding "padded-blocks": 2, https://gist.github.com/ntwb/2c9bb14d91e1a79137a1e90e04256493

from eslint-config-stylelint.

@ntwb i think never options is more good, than always (remove extra blank lines)

from eslint-config-stylelint.

I updated the gist with using the never option, much saner ~900 line diff, though I'm still undecided if this is needed though....

https://gist.github.com/ntwb/2c9bb14d91e1a79137a1e90e04256493

from eslint-config-stylelint.

@ntwb what about size of package, before and after?

from eslint-config-stylelint.

Does anyone have a strong opinion on whether the "never" option makes the code more readable or not?

If there are no objections then I'm happy to see this go in.

from eslint-config-stylelint.

I'm fine with this.

from eslint-config-stylelint.

@ntwb what about size of package, before and after?

It would be quite minor, the change removes 81 blank lines

Does anyone have a strong opinion on whether the "never" option makes the code more readable or not?

I think it does, but that's only because I use a similar standard with the PHP code I write, so it works for me

from eslint-config-stylelint.

Done in #10

from eslint-config-stylelint.