phpinsights icon indicating copy to clipboard operation
phpinsights copied to clipboard

Fixes #553 adds dont-exclude-tests option to allow analyse of tests folders

Open alexmakii opened this issue 1 year ago • 3 comments

Added dont_exclude_tests config

  • Added config option "dont_exclude_tests"
  • Added cli option "--dont-exclude-tests" Small refactor of LocalFilesRepository
  • Removed hardcoded list of files for exclusion
  • Removed explicit exclussion of files by name, are already excluded by path Added docs & updated tests
Q A
Bug fix? no
New feature? yes
Fixed tickets #553

alexmakii avatar Jul 31 '22 16:07 alexmakii

@JustSteveKing Can I do anything else to make this PR merged ?

This feature doesn't add breaking changes.

alexmakii avatar Aug 06 '22 15:08 alexmakii

@alexmakii just running actions, will merge if all good!

JustSteveKing avatar Aug 06 '22 15:08 JustSteveKing

@alexmakii this seems to have introduced a failing test, as soon as this is passing I would be happy to merge this in - it's been quite a requested feature!

JustSteveKing avatar Aug 06 '22 15:08 JustSteveKing

Closing until tests and conflicts are resolved

cmgmyr avatar Sep 28 '23 02:09 cmgmyr