Add a new rule to make sure function parens are consistent.

[aljones15]

This means the following

invalid:

before(async () => {
  }
) // error here

valid:

function multiArgs(
  one, two,
  three, four,
  five, six
) {

}

p.s. because this will almost 100% lead to multiple different rule set ideas you can see all the ideas here:

https://eslint.org/docs/rules/function-paren-newline

p.s. no issue if you want error never I can refactor any function that takes more than say 3 args to take a single named object.

[mattcollier]

@aljones15 don't forget to add a changelog entry which makes the release easier.