You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description
When specifying paths in the command, the excludes configured in the config file is ignored.
Use case
We run PHP insights on PRs, but we only want to run the insights on the files that has been changed in the PR and not the whole project. We use PHP Insights in a GitHub Action with the following configuration to accomplish this:
The problem is that we specify a few paths that should be in our excluded in out configuration, but this seems to be ignored when the files are passed to the insights command. I think they should be excluded according to the config even if you specify them explicitly in the command.
Maybe it would be good to add an option for this to avoid breaking the existing behavior?
We use Laravel 10.13.5
The text was updated successfully, but these errors were encountered:
Description
When specifying paths in the command, the excludes configured in the config file is ignored.
Use case
We run PHP insights on PRs, but we only want to run the insights on the files that has been changed in the PR and not the whole project. We use PHP Insights in a GitHub Action with the following configuration to accomplish this:
The problem is that we specify a few paths that should be in our excluded in out configuration, but this seems to be ignored when the files are passed to the insights command. I think they should be excluded according to the config even if you specify them explicitly in the command.
Maybe it would be good to add an option for this to avoid breaking the existing behavior?
We use Laravel
10.13.5
The text was updated successfully, but these errors were encountered: