Skip to content

Conversation

@Namoshek
Copy link
Contributor

Currently, two different multi-line constructor formats are used. This PR changes the format to a common one (which is used more frequently throughout the code base already).

Not very common:

public function __construct(protected SwooleExtension $extension,
protected $timerTable,
protected $maxExecutionTime)
{
}

Common:

public function __construct(
protected SignalDispatcher $dispatcher,
protected ServerStateFile $serverStateFile
) {
}

@taylorotwell taylorotwell merged commit bc203d8 into laravel:master Apr 11, 2021
@Namoshek Namoshek deleted the fix-code-formatting branch April 11, 2021 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants