Thanks, Chi! I haven't had a chance to experience yet how these tools work with PRs, but will keep a look out for how useful they seem. A quick scan through the LGTM report of master shows that it found some real issues and isn't too noisy. There's a spread of false positives (it doesn't like loop unrolling), borderline cases, and legitimate issues that would be great to fix.
On Tue, Aug 20, 2019 at 4:56 PM Chi Cao Minh <chi.caom...@imply.io> wrote: > Coveralls.io <http://coveralls.io/> and LGTM.com <http://lgtm.com/> > GitHub integrations are now enabled to help us improve code quality, and > you’ll start seeing updates on pull requests from the respective bots. > Also, for code coverage, JaCoCo is now used instead of Cobertura. > > Druid’s Coveralls.io <http://coveralls.io/> page: > https://coveralls.io/github/apache/incubator-druid < > https://coveralls.io/github/apache/incubator-druid> > > Druid’s LGTM.com <http://lgtm.com/> page: > https://lgtm.com/projects/g/apache/incubator-druid/ < > https://lgtm.com/projects/g/apache/incubator-druid/> > > If you have feedback on either of the new GitHub integrations, please > reply to this thread. > > Thanks, > Chi > >
