Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable cobertura travis portion #3122

Merged
merged 1 commit into from
Jun 13, 2016

Conversation

drcrallen
Copy link
Contributor

@drcrallen drcrallen commented Jun 10, 2016

@gianm
Copy link
Contributor

gianm commented Jun 10, 2016

@drcrallen what signs are there that this step is causing the stalls from #3067?

@drcrallen
Copy link
Contributor Author

Every time I've checked that a failure for the bug in question it is on that step. You scroll back up the log to see what is running.

@gianm
Copy link
Contributor

gianm commented Jun 10, 2016

ok, fair enough.

👍

@nishantmonu51
Copy link
Member

👍

@nishantmonu51 nishantmonu51 merged commit cf90508 into apache:master Jun 13, 2016
@drcrallen drcrallen deleted the disableCobertura branch June 13, 2016 18:22
@gianm gianm modified the milestones: 0.9.2, 0.9.1 Sep 23, 2016
@ccaominh ccaominh mentioned this pull request Aug 15, 2019
1 task
ccaominh added a commit to ccaominh/druid that referenced this pull request Aug 16, 2019
Code coverage was disabled via
apache#3122 due to an issue with
cobertura in Travis CI. Switch code coverage tool from cobertura to
jacoco to avoid issue and re-enable coveralls for Travis CI.
gianm pushed a commit that referenced this pull request Aug 20, 2019
* Enable code coverage

Code coverage was disabled via
#3122 due to an issue with
cobertura in Travis CI. Switch code coverage tool from cobertura to
jacoco to avoid issue and re-enable coveralls for Travis CI.

* Exclude non-production code

* Exclude benchmark generated code

* Exclude DruidTestRunnerFactory
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants