Rule suggestion: Enforce padding between class functions

[LinusU]

I would like to propose a rule that enforces consistent spacing between class functions.

The following patterns would be considered problems:

class Test1 {
  constructor () {
    // ...
  }
  otherFunc () {
    // ...
  }
}

class Test2 {
  constructor () {
    // ...
  }


  otherFunc () {
    // ...
  }
}

The following pattern would be considered valid:

class Test3 {
  constructor () {
    // ...
  }

  otherFunc () {
    // ...
  }
}

[yoshuawuyts]

sure, seems reasonable

[feross]

There's no eslint core rule for this? @LinusU can you propose it on the eslint repo?

[LinusU]

eslint/eslint#5949

[LinusU]

I also created a working rule 🙌

eslint/eslint#5950

[feross]

@LinusU Nice work! Please post a comment here / send a PR when that's merged and released in ESLint so we can add this rule to standard.

[LinusU]

Will do 👌

[feross]

@LinusU Looks like progress is going well on this rule -- it's almost there! I'm going to close this issue since it's already being tracked in the ESLint repo and there's nothing for us to do here.

Please open a new issue or PR when this rule is actually added to ESLint and I'll enable it. If you forget, no worries either. Every time I update ESLint, I evaluate every new rule for inclusion in standard so I will see it.

[LinusU]

eslint/eslint@ee99876

It has finally landed 🚀 🎉

[LinusU]

Landed in eslint 4.9.0 🎉

[feross]

Let's include this in standard 13.

Ecosystem impact: only one failing package (webtorrent) and was automatically fixable.