-
-
Notifications
You must be signed in to change notification settings - Fork 25
Out of memory errors #4
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.
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
Comments
It seems to be a php memory limit as you thought. e.g. Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /usr/local/lib/php-code-quality/vendor/phploc/phploc/src/Analyser.php on line 92 Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /usr/local/lib/php-code-quality/vendor/phploc/phploc/src/Analyser.php on line 209 Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /usr/local/lib/php-code-quality/vendor/phploc/phploc/src/Analyser.php on line 241 |
Looking at the Dockerfile, it was set at 512M. |
Thanks @hackzilla that's worked for me. I wonder if the commands in the readme should be updated to reflect this. |
Great idea. Also, I'll be updating to use PHP 7.2 to prevent the 7.3 deprecations that affect PHPLoc. |
I added a note to the README for future users to know how to handle memory limit errors. |
Full disclosure: This is my first time trying Docker, after seeing your excellent talk at PHPUK19.
I have run two commands and both have written an out of memory exception to the resulting file.
$ docker run -it --rm -v "$PWD":/app -w /app adamculp/php-code-quality:latest php /usr/local/lib/php-code-quality/vendor/bin/phploc -v --names "*.php" --exclude "vendor" . > ./phploc.txt
Results in a file containing
When running the
phpcpd
command the file results inIs this related to my locally installed PHP ini settings? Any guidance would be super. Thanks!
The text was updated successfully, but these errors were encountered: