Skip to content

tools: make trailing comma lint rule consistent

Rodrigo Muino Tomonari requested to merge github/fork/Trott/function-trailing-commas into master

I don't know how I feel about trailing commas in function definitions and invocations, but let's at least make it consistent with our allowed use of trailing commas elsewhere and restrict it to multi-line situations. ESLint defaults to ignoring trailing commas in function definitions and invocations. This change will make it an error to have a trailing comma if the statement is a single line. For multiline statements, a trailing comma is permitted but is not required.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports