sijie commented on issue #246: Issue 245 Enable Coverall.io and Covertura on Travis configuration file URL: https://github.com/apache/bookkeeper/pull/246#issuecomment-315854380 a few comments: 1) currently we only used travis ci for verifying pull requests (e.g. findbugs and checkstyles) 2) if you want to run coverage for each pull requests, you can skip tests for now. but I am not sure if we need to run coverage for every pull requests. I would suggest starting with master. 3) coveralls requires a token to report correctly. you might need to setup a secret env variable in travis ci. or it might be making sense just do this in jenkins. (DL does coverage on jenkins only) ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
With regards, Apache Git Services
