-
-
Notifications
You must be signed in to change notification settings - Fork 283
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
Add --allow-traits option #1
Comments
I am curious why traits are not allowed by default? At any rate, if they are indeed disabled by default, then providing Traits 1 ✘ --> The use of `traits` is prohibited:
• app/FilterTrait.php |
This is already possible with the configuration file. |
Good point that the config allows for the same over-rides as base php insights, but it'd be harmless to make this point of customisation easier / clearer (even if just in the docs or this issue). Especially as Traits are awesome and widely used in Laravel packages. For anyone looking for the solution (via google) the import (into your insights.php) to suppress the Trait warning is: NunoMaduro\PhpInsights\Domain\Insights\ForbiddenTraits; |
vendor/bin/phpinsights analyse --allow-traits
The text was updated successfully, but these errors were encountered: