Re: Add Errorprone to build process?
For findbugs we suppressed ~everything and filed bugs, so that might also be a way to get started to get errorprone on the main build. Another framework worth looking at is the Checker framework, which has analysis mathematically proven to always prevent all NPEs. So that would replace the most important (IMO) use of findbugs with greater rigor. (it is actually not about specifically NPEs being a big danger, but about elevating the level of our code to "no NPEs possible" style, as it is more readable, maintainable, flexible) Kenn On Tue, Feb 13, 2018 at 2:15 PM, Ismaël Mejía wrote: > The approach so far is optional (behind a profile), iin any case > contributions are more than welcome to fix the error-prone detected > issues once this is merged. > > On Tue, Feb 13, 2018 at 10:03 PM, Eugene Kirpichov > wrote: > > Oh! I had no idea this was already in progress - thanks Kenn! > > > > > > On Tue, Feb 13, 2018, 12:23 PM Ismaël Mejía wrote: > >> > >> Kenn submitted a PR for this yesterday. So I assume is taken. > >> https://github.com/apache/beam/pull/4667 > >> > >> On Tue, Feb 13, 2018 at 8:49 PM, Eugene Kirpichov > > >> wrote: > >> > Filed https://issues.apache.org/jira/browse/BEAM-3697 > >> > > >> > This seems to be a good task for a new contributor, easy and > potentially > >> > with high payoff (uncovering bugs), and a good way to become cursorily > >> > familiar with diverse parts of the codebase by fixing the bugs it > finds. > >> > > >> > Any takers? > >> > > >> > Description: > >> > > >> > http://errorprone.info/ is a good static checker that covers a > number of > >> > bugs not covered by FindBugs or Checkstyle. We use it internally at > >> > Google > >> > and, when run on the Beam codebase, it occasionally uncovers issues > >> > missed > >> > during PR review process. > >> > > >> > It has Maven and Gradle plugins: > >> > http://errorprone.info/docs/installation > >> > https://github.com/tbroyer/gradle-errorprone-plugin > >> > > >> > It would be good to integrate it into our Maven and Gradle builds. >
Re: Add Errorprone to build process?
The approach so far is optional (behind a profile), iin any case contributions are more than welcome to fix the error-prone detected issues once this is merged. On Tue, Feb 13, 2018 at 10:03 PM, Eugene Kirpichov wrote: > Oh! I had no idea this was already in progress - thanks Kenn! > > > On Tue, Feb 13, 2018, 12:23 PM Ismaël Mejía wrote: >> >> Kenn submitted a PR for this yesterday. So I assume is taken. >> https://github.com/apache/beam/pull/4667 >> >> On Tue, Feb 13, 2018 at 8:49 PM, Eugene Kirpichov >> wrote: >> > Filed https://issues.apache.org/jira/browse/BEAM-3697 >> > >> > This seems to be a good task for a new contributor, easy and potentially >> > with high payoff (uncovering bugs), and a good way to become cursorily >> > familiar with diverse parts of the codebase by fixing the bugs it finds. >> > >> > Any takers? >> > >> > Description: >> > >> > http://errorprone.info/ is a good static checker that covers a number of >> > bugs not covered by FindBugs or Checkstyle. We use it internally at >> > Google >> > and, when run on the Beam codebase, it occasionally uncovers issues >> > missed >> > during PR review process. >> > >> > It has Maven and Gradle plugins: >> > http://errorprone.info/docs/installation >> > https://github.com/tbroyer/gradle-errorprone-plugin >> > >> > It would be good to integrate it into our Maven and Gradle builds.
Re: Add Errorprone to build process?
Oh! I had no idea this was already in progress - thanks Kenn! On Tue, Feb 13, 2018, 12:23 PM Ismaël Mejía wrote: > Kenn submitted a PR for this yesterday. So I assume is taken. > https://github.com/apache/beam/pull/4667 > > On Tue, Feb 13, 2018 at 8:49 PM, Eugene Kirpichov > wrote: > > Filed https://issues.apache.org/jira/browse/BEAM-3697 > > > > This seems to be a good task for a new contributor, easy and potentially > > with high payoff (uncovering bugs), and a good way to become cursorily > > familiar with diverse parts of the codebase by fixing the bugs it finds. > > > > Any takers? > > > > Description: > > > > http://errorprone.info/ is a good static checker that covers a number of > > bugs not covered by FindBugs or Checkstyle. We use it internally at > Google > > and, when run on the Beam codebase, it occasionally uncovers issues > missed > > during PR review process. > > > > It has Maven and Gradle plugins: > > http://errorprone.info/docs/installation > > https://github.com/tbroyer/gradle-errorprone-plugin > > > > It would be good to integrate it into our Maven and Gradle builds. >
Re: Add Errorprone to build process?
Kenn submitted a PR for this yesterday. So I assume is taken. https://github.com/apache/beam/pull/4667 On Tue, Feb 13, 2018 at 8:49 PM, Eugene Kirpichov wrote: > Filed https://issues.apache.org/jira/browse/BEAM-3697 > > This seems to be a good task for a new contributor, easy and potentially > with high payoff (uncovering bugs), and a good way to become cursorily > familiar with diverse parts of the codebase by fixing the bugs it finds. > > Any takers? > > Description: > > http://errorprone.info/ is a good static checker that covers a number of > bugs not covered by FindBugs or Checkstyle. We use it internally at Google > and, when run on the Beam codebase, it occasionally uncovers issues missed > during PR review process. > > It has Maven and Gradle plugins: > http://errorprone.info/docs/installation > https://github.com/tbroyer/gradle-errorprone-plugin > > It would be good to integrate it into our Maven and Gradle builds.
Add Errorprone to build process?
Filed https://issues.apache.org/jira/browse/BEAM-3697 This seems to be a good task for a new contributor, easy and potentially with high payoff (uncovering bugs), and a good way to become cursorily familiar with diverse parts of the codebase by fixing the bugs it finds. Any takers? Description: http://errorprone.info/ is a good static checker that covers a number of bugs not covered by FindBugs or Checkstyle. We use it internally at Google and, when run on the Beam codebase, it occasionally uncovers issues missed during PR review process. It has Maven and Gradle plugins: http://errorprone.info/docs/installation https://github.com/tbroyer/gradle-errorprone-plugin It would be good to integrate it into our Maven and Gradle builds.