kykrueger commented on PR #1806: URL: https://github.com/apache/systemds/pull/1806#issuecomment-1518593020
@Baunsgaard I've added the directory of the jacoco report as an artifact at the end of the java test workflow. The Job cleans up all of the artifacts from the steps in between to avoid billing problems with the number of artifacts. I'd suggest we just keep the csv from the export to keep the artifact size down, but now we need to agree upon the metric for a test coverage badge. What would you think of just having one with the lines covered in percentage? It should also be possible to add some regression checks, by comparing the percentage covered in the branch being merged with the amount covered in main and failing the PR if the percentage drops. Perhaps that would be a good separate ticket though, I don't want to blow up the scope of this one. -- 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. To unsubscribe, e-mail: dev-unsubscr...@systemds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
