You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Mark Bretl <mb...@pivotal.io> on 2015/09/25 00:32:05 UTC

Review Request 38737: GEODE-272: Change Findbugs reportLevel to 'low'

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38737/
-----------------------------------------------------------

Review request for geode.


Bugs: GEODE-372
    https://issues.apache.org/jira/browse/GEODE-372


Repository: geode


Description
-------

The build currently sets the default Findbugs report level to 'high'. According to the plugin documentation, this means it will only report the 'high' level warnings.

The default reporting level is 'medium', which means medium and high are reported.


Diffs
-----

  build.gradle d7b4965 

Diff: https://reviews.apache.org/r/38737/diff/


Testing
-------

./gradlew clean findbugsMain

Verified low, medium, and high bugs are reported


Thanks,

Mark Bretl


Re: Review Request 38737: GEODE-372: Change Findbugs reportLevel to 'low'

Posted by Kirk Lund <ki...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38737/#review100857
-----------------------------------------------------------

Ship it!


Ship It!

- Kirk Lund


On Sept. 24, 2015, 10:32 p.m., Mark Bretl wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38737/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2015, 10:32 p.m.)
> 
> 
> Review request for geode.
> 
> 
> Bugs: GEODE-372
>     https://issues.apache.org/jira/browse/GEODE-372
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> The build currently sets the default Findbugs report level to 'high'. According to the plugin documentation, this means it will only report the 'high' level warnings.
> 
> The default reporting level is 'medium', which means medium and high are reported.
> 
> 
> Diffs
> -----
> 
>   build.gradle d7b4965 
> 
> Diff: https://reviews.apache.org/r/38737/diff/
> 
> 
> Testing
> -------
> 
> ./gradlew clean findbugsMain
> 
> Verified low, medium, and high bugs are reported
> 
> 
> Thanks,
> 
> Mark Bretl
> 
>


Re: Review Request 38737: GEODE-372: Change Findbugs reportLevel to 'low'

Posted by Mark Bretl <mb...@pivotal.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38737/
-----------------------------------------------------------

(Updated Sept. 24, 2015, 10:32 p.m.)


Review request for geode.


Changes
-------

Updated Summary to correct bug


Summary (updated)
-----------------

GEODE-372: Change Findbugs reportLevel to 'low'


Bugs: GEODE-372
    https://issues.apache.org/jira/browse/GEODE-372


Repository: geode


Description
-------

The build currently sets the default Findbugs report level to 'high'. According to the plugin documentation, this means it will only report the 'high' level warnings.

The default reporting level is 'medium', which means medium and high are reported.


Diffs
-----

  build.gradle d7b4965 

Diff: https://reviews.apache.org/r/38737/diff/


Testing
-------

./gradlew clean findbugsMain

Verified low, medium, and high bugs are reported


Thanks,

Mark Bretl