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

(Updated Oct. 17, 2016, 12:25 p.m.)


Review request for oozie and Robert Kanter.


Changes
-------

Going for the `v0.1.0-all` package from Maven Central instead of GitHub.


Repository: oozie-git


Description
-------

We go for that solution:

1. create a new script as `bin/test-patch-11-findbugs-diff` so that it comes 
right after compile

2. `mvn clean verify -DskipTests` before and after applying the patch, and 
saving each and every module's `target/findbugs/findbugsXml.xml` to a secure 
folder

3. call the `findbugs-diff` app for all the before / after FindBugs xml pairs

4. analyze all the `findbugs-new.xml` entries for nodes with XPath 
`/BugCollection/BugInstance` entries. Stop on any BugInstance w/ `priority` and 
/ or `rank` lower than threshold

5. have the reports merged and be downloadable in any case in following 
scenarios:

* -1 for bugs that are too serious to apply the patch with them
* 0 for other new bugs
* +1 if there is no new bug


Diffs (updated)
-----

  bin/test-patch-11-findbugs-diff PRE-CREATION 
  bin/test-patch-11-findbugs-diff-0.1.0-all.jar.md5 PRE-CREATION 
  pom.xml 83a1d42 

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


Testing
-------


Thanks,

András Piros

Reply via email to