accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-2529) bake desired warnings into normal maven build
Date Fri, 18 Apr 2014 16:48:15 GMT


Christopher Tubbs commented on ACCUMULO-2529:

Doesn't appear possible to fail the build with the default plugin set. Not sure it's worth
adding any other maven plugins to do that. Also, I'm not sure it's worth it. It's sufficient
to see them, I think. I'd treat these like any other warning from findbugs or something else....
something to address, and we'd prefer to avoid coding practices that introduce new ones (if
we can catch them), but not necessarily something that is so bad it should fail the build,
as these typically require manual triage.

> bake desired warnings into normal maven build
> ---------------------------------------------
>                 Key: ACCUMULO-2529
>                 URL:
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: build
>    Affects Versions: 1.4.5, 1.5.1, 1.6.0
>            Reporter: Sean Busbey
>            Assignee: Christopher Tubbs
>             Fix For: 1.4.6, 1.5.2, 1.6.1, 1.7.0
> We've needed several follow on tickets lately to fix warnings introduced by patches.
> It would be easier to catch these earlier if we update our maven configuration to surface
them in the normal build.
> * We should show warnings for lint issues, generally
> * If possible, we should fail the build if issues show up
> (If the current tree isn't clean, a follow on cleanup ticket should be made a prereq
for this one)

This message was sent by Atlassian JIRA

View raw message