You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@bookkeeper.apache.org by gi...@git.apache.org on 2017/07/17 19:18:41 UTC

[GitHub] sijie commented on issue #246: Issue 245 Enable Coverall.io and Covertura on Travis configuration file

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:
users@infra.apache.org


With regards,
Apache Git Services