maytasm edited a comment on pull request #10171: URL: https://github.com/apache/druid/pull/10171#issuecomment-661330263
> > The code coverage bot doesn't like missing tests for the classes I added for testing. Those classes were added in `integration-tests` module, and I think we can ignore this report. > > I think we can ignore all classes within the `integration-tests` module from the code coverage check. It looks like we can add that exclusion around here - > > https://github.com/apache/druid/blob/2b48de074a855d3c36ff588e90846644fc4b5f42/.travis.yml#L168 > > If we need another patch, can you add the exclusion, if the patch is good to go, I'm ok with adding the exclusion in another PR. Which job did it failed on? I thought the integration-tests module does not have any job with the code coverage running ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
