eiri commented on issue #852: Enable coverage reports for more applications URL: https://github.com/apache/couchdb/pull/852#issuecomment-333181450 It's running for couch_epi and couch_log, I believe, for both because they've been build from a scratch, test cover was usable for development and then got included as a part of a bigger merge, not because it was turned on exclusively. We don't know how much overhead that adds, because we never measured it :) My point thought not about performance penalty per se, but about unnecessary movement. In CI the generated reports not visible and not easily accessible, apart from scrolling the endless log, so it's not really helpful to have them always on. In general I'm +0 on this change. If you get +1 from someone I wouldn't block it, our test suite already bulky and verbose, adding coverage not going to change things dramatically. I don't know easy way to make rebar switch cover, sorry :/ In rebar3 it is a separate command, which makes sense for me, but our rebar, I'm afraid, would require script trickery. ---------------------------------------------------------------- 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: [email protected]
With regards, Apache Git Services
