Skip to content
This repository has been archived by the owner on Jan 6, 2020. It is now read-only.

Fix PSR-2 line length recommendations #454

Merged
merged 1 commit into from
Dec 18, 2015
Merged

Fix PSR-2 line length recommendations #454

merged 1 commit into from
Dec 18, 2015

Conversation

eliecharra
Copy link
Contributor

Extension template is generating an inline phpdoc @link which exceed PSR-2 line lenght recommendation.

http://www.php-fig.org/psr/psr-2/#1-overview

There MUST NOT be a hard limit on line length; the soft limit MUST be 120 characters; lines SHOULD be 80 characters or less.

@link anotation is enough imho, maybe previous text could be removed to get under recommended line length.

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no

@fabpot
Copy link
Member

fabpot commented Dec 18, 2015

Thank you @eliecharra.

@fabpot fabpot merged commit d83ed9f into sensiolabs:master Dec 18, 2015
fabpot added a commit that referenced this pull request Dec 18, 2015
This PR was merged into the 3.0.x-dev branch.

Discussion
----------

Fix PSR-2 line length recommendations

Extension template is generating an inline phpdoc `@link` which exceed PSR-2 line lenght recommendation.

http://www.php-fig.org/psr/psr-2/#1-overview

> There MUST NOT be a hard limit on line length; the soft limit MUST be 120 characters; lines SHOULD be 80 characters or less.

`@link` anotation is enough imho, maybe previous text could be removed to get under recommended line length.

 Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no

Commits
-------

d83ed9f Fix PSR2 line length recommendations
@eliecharra
Copy link
Contributor Author

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants