-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
feature request: option to exclude directories from analysis. #156
Comments
Do we have to reinvent everything that PHPStan does? PHPStan has a convenient way to ignore errors: https://phpstan.org/user-guide/ignoring-errors Can we run drupal-check somehow in PHPStan mode with all of its behavior except that the drupal-checks are also executed? |
Ah cool, this should be possible with https://github.com/mglaman/phpstan-drupal ! Will try that next. |
#156: Added an exclude option to prevent analysis in selected directories.
@klausi want to open an issue around that topic? Basically drupal-check is a simple runner for folks who don't want to configure PHPStan. |
We have Codeception acceptance tests in a D8 profile, and when we try to use the
drupal-check
tool to run analysis on the profile, we're finding it is throwing errors like:It'd be super if we could exclude those directories via the command invocation:
drupal-check docroot/profiles/custom --exclude-dir=*/tests/codeception
Likely, it'd just be a matter of looking for the option and adding the directory pattern to here:
drupal-check/src/Command/CheckCommand.php
Line 129 in 3ec4b60
If that's something you'd be open to, I can probably make a PR fairly quickly.
The text was updated successfully, but these errors were encountered: