[ 
https://issues.apache.org/jira/browse/YETUS-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15712105#comment-15712105
 ] 

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:
{code}
    if [[ ! -d "${MODULE[${i}]}/src/main/java"
       && ! -d "${MODULE[${i}]}/src/test/java" ]]; then
       skiplist=("${skiplist[@]}" "${MODULE[$i]}")
    fi
{code}

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
(v6.3.4#6332)

Reply via email to