Skip to content

Commit 398cadc

Browse files
authored
Merge pull request #34 from shoppingflux/issue-#26
#26 Add requirements for line jump
2 parents 019192f + 71c4ad9 commit 398cadc

File tree

1 file changed

+23
-0
lines changed

1 file changed

+23
-0
lines changed

docs/style.md

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,29 @@ $this->object
8585
->method4();
8686
```
8787

88+
### Method line jump declaration
89+
90+
Do not arbitrary jump lines in method declaration : Only jump lines when max line chars is reached.
91+
92+
##### Correct
93+
94+
```php
95+
public function handle(ServerRequestInterface $request, ServerHandlerInterface $handler): void
96+
{
97+
}
98+
```
99+
##### Not correct
100+
101+
```php
102+
// Max line chars was not reached, we should not jump lines between params
103+
public function handle(
104+
ServerRequestInterface $request,
105+
ServerHandlerInterface $handler
106+
): void
107+
{
108+
}
109+
```
110+
88111
### Class naming
89112

90113
If your class use a type, a role or a pattern, you should declare it in the class name.

0 commit comments

Comments
 (0)