appstract / laravel-opcache

Laravel Package for OPcache
MIT License
1.34k stars 119 forks source link

Add more paths to config's exclude array #91

Closed 0x4r45h closed 5 years ago

0x4r45h commented 5 years ago

In a fresh laravel 6.x project running php artisan opcache:compile --force throws exception due to existence of a file called vendor/symfony/var-dumper/Tests/Fixtures/Php74.php adding Tests to exclude array fixes the problem. in addition as the exclude function is used at the moment IS case-sensitive and on the other hand there is not any standard convention for third party libraries to name their tests/stubs folder,in a real life project i found all these variants naming in libraries obviously there is no point to put test folders in opcache so i think trying to exclude as much as test files by default is a good practice