| In 2.x branch, we have findbugs check on core/impl which is pretty much sufficient as the whole codebase resides there. In 3.x the check remains, yet we have modules (JTA, EJB, JSF, Web) which are therefore excluded. I believe we should include them - will send a PR with that. I was also thinking about checking Probe this way (neither branch does that ATM). Do you think it makes sense? From my pov it's "optional". |