Problem/Motivation

To enforce the standard approved in #3295249: Allow multi-line function declarations.

Some testing was done for core about this that should be helpful. The results are in #3295249-57: Allow multi-line function declarations

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

quietone created an issue. See original summary.

  • klausi authored 0c6ba2c2 on 8.3.x 
    feat(MultiLineFunctionDeclaration): Enable multi-line function...
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Fixed

Makes sense, I pushed the configuration change to validate the indentation of mulit-line function declaration.

Missing is the trailing comma validation for the last parameter, did not find anything in the upstream Squiz.Functions.MultiLineFunctionDeclaration sniff. Will open a new issue for that.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented

Created #3440603: Validate trailing comma in multi-line function declaration.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.