🐛 Skip "target" folder in file analysis

[aufi]

Updating filecontent condition to exclude files in "target" subdirectory. This condition already used grep, so adding --exclude-dir option there.

Fixes #358

[aufi]

@jmle I'm not sure such simple fix is appropriate here, thinking about listing target directories first and check they are not in git, so were likely generated recently and should be skipped. (and if there is no git repo, skip target automatically or do some changed timestamp check?)

[jmle]

Hey @aufi, sorry I somehow missed this one last month. The main problem I see with this approach is that the filecontent condition is in the internal provider, and therefore not exclusive to Java, but the target problem will only happen in Java. I'm also wondering if this problem is exclusive to the filecontent condition, or could also appear for instance with the javaclass conditions.

[shawn-hurley]

@aufi Hey,

So I am going to close this, it was fixed in a different way in PR #534

Please let me know if you think that this is not correct or if this fixes something else and we can reopen

[aufi]

#534 covers this issue, so no problem with closing this PR, thanks Shawn!