lorisleiva / laravel-docker

🐳 Generic docker image for Laravel Applications
MIT License
934 stars 314 forks source link

Alternative install method for PHP_CodeSniffer #15

Closed fgilio closed 5 years ago

fgilio commented 5 years ago

Hello! I maintain a fork of this image that adds yarn. Yesterday I noticed the image builds where failing and the problem was installing PHP_CodeSniffer, so I made this changes to install it using Composer instead of PEAR. I guess this might be related to the site being down http://pear.php.net/

Here's a screenshot of the error I was getting: screen shot 2019-02-06 at 11 34 09

EDIT: libzip is required or the build fails while installing PHP_CodeSniffer

lorisleiva commented 5 years ago

Amazing, fixing that failed build was on To-Do list. Thank you very much for that! Btw, feel free to make a PR for adding yarn. 👍

fgilio commented 5 years ago

I'm glad it helps! yarn PR in a few minutes