Peter Vary commented on YETUS-471:

These were the ones used before my change, so it felt proper to keep them thus 
keeping the change backward compatible.
See the original code:
    if [[ ! -d "${MODULE[${i}]}/src/main/java"
       && ! -d "${MODULE[${i}]}/src/test/java" ]]; then
       skiplist=("${skiplist[@]}" "${MODULE[$i]}")

By the way these are the maven default source which are usually kept in most of 
the maven projects (but of course not in Hive :) )

> Add possibility to configure Findbugs source folders
> ----------------------------------------------------
>                 Key: YETUS-471
>                 URL: https://issues.apache.org/jira/browse/YETUS-471
>             Project: Yetus
>          Issue Type: Improvement
>          Components: Test Patch
>    Affects Versions: 0.3.0
>            Reporter: Peter Vary
>            Assignee: Peter Vary
>         Attachments: YETUS-471.00.patch, YETUS-471.01.patch
> Trying to add yetus as a pre-commit test for Hive (see HIVE-15051)
> Findbugs plugin currently expects the project structure to be the standard 
> maven one (/src/main/java, /src/test/java). In Hive some of the modules have 
> a different file layout. It would be good to have a parameter introduced to 
> enable changes in checking directories.

This message was sent by Atlassian JIRA

Reply via email to