Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tools: lint for function argument alignment #6390

Closed
wants to merge 2 commits into from
Closed

tools: lint for function argument alignment #6390

wants to merge 2 commits into from

Commits on Apr 27, 2016

  1. test,tools: adjust function argument alignment 

    In preparation for a lint rule enforcing function argument alignment,
adjust function arguments to be aligned.
    @Trott
    Trott committed Apr 27, 2016
    Configuration menu
    Copy the full SHA
    0c69eb9 View commit details
    Browse the repository at this point in the history

  2. tools: lint for function argument alignment 

    In function calls that span multiple lines, apply a custom lint rule to
enforce argument alignment.

With this rule, the following code will be flagged as an error by the
linter because the arguments on the second line start in a different
column than on the first line:

    myFunction(a, b,
      c, d);

The following code will not be flagged as an error by the linter:

    myFunction(a, b,
               c, d);
    @Trott
    Trott committed Apr 27, 2016
    Configuration menu
    Copy the full SHA
    4b0c0d8 View commit details
    Browse the repository at this point in the history