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

Do not use line breaks in method definitions #15

Merged

ausi merged 1 commit into develop from hotfix/more-cs

Aug 22, 2016

Member

leofeyer commented Aug 9, 2016

No description provided.


          Do not use line breaks in method definitions.

a71f8c0

ausi merged commit b76fe30 into develop

Member

ausi commented Aug 22, 2016

;( Maybe we should switch to a sane coding standard one day...

Member

ausi commented Aug 22, 2016

And it looks like scrutinizer doesn’t like it: https://scrutinizer-ci.com/g/contao/image/inspections/df0a7be3-5fe8-48bc-934b-ece4df9e647d/issues/files/src/PictureGenerator.php?status=new&selectedLabels%5B0%5D=3&orderField=path&order=asc&honorSelectedPaths=0

Member Author

leofeyer commented Aug 22, 2016

I already told you to adjust the .scrutinizer.yml file and set max_length: '120' to max_length: '0'.

Member

ausi commented Aug 23, 2016

Ah, missed that. Fixed in 4ff783a

ausi deleted the hotfix/more-cs branch

August 23, 2016 07:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment