Feature/exclude directory for insights

[Jibbarth]

Q	A
Bug fix?	no
New feature?	yes
Fixed tickets	#240 #243 #108 (comment)

This PR add possibility to configure exclude part with a directory in addition of filepath:

        ForbiddenSetterSniff::class => [
            'exclude' => [
                'src/Domain/MySpeciedClass',
                'src/Models',
            ],
        ],

In the example above, all classes in src/Models will be excluded for the ForbiddenSetterSniff

This PR also port the exclude feature to our own Insights (ForbiddenDefineGlobalConstants, ForbiddenGlobals ...). I added the exclude check for each Insights that implement HasDetail interface.

[olivernybroe]

I have commented on some small improvements for readability and performance :)

[olivernybroe]

We should maybe consider changing our preset after this gets merged. I think we could help the Laravel developers at least with some changes here.

[olivernybroe]

Alright, I think this is almost ready now.

There is a duplication on a method which I think we should handle, either with a trait, a helper class or just a adding the method to our insight class.

[olivernybroe]

How does this act in case where getcwd returns false?

[Jibbarth]

I tried and it throw a DirectoryNotFoundException.

I can add a check on $basedir in Files::find and return an empty array if it's not exist to avoid throw an exception ? WDYT

[olivernybroe]

Hmm whatever you prefer here tbh. I don't really have an opinion

[olivernybroe]

Alright, when you have figured out which way you want to handle that weird edgecase with cwd, then you can go ahead 👍

The code is really nice, great work! 😃

[Jibbarth]

@olivernybroe I add a fallback when getcwd return false to use configuration Directory 😉