Skip to content
This repository was archived by the owner on May 30, 2024. It is now read-only.

Commit 2d0628d

Browse files
committed
better instructions
1 parent 3fad450 commit 2d0628d

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

CONTRIBUTING.md

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,6 @@ To build the SDK and run all unit tests:
4545

4646
## Code coverage
4747

48-
It is important to keep unit test coverage as close to 100% as possible in this project.
48+
It is important to keep unit test coverage as close to 100% as possible in this project. You can view the latest code coverage report in CircleCI, as `coverage/html/index.html` in the artifacts for the "Java 11 - Linux - OpenJDK" job. You can also run the report locally with `./gradlew jacocoTestCoverage` and view `./build/reports/jacoco/test`.
4949

50-
Sometimes a gap in coverage is unavoidable, usually because the compiler requires us to provide a code path for some condition that in practice can't happen and can't be tested, or because of a known issue with the code coverage tool. Please handle all such cases as follows:
51-
52-
* Mark the code with an explanatory comment beginning with "COVERAGE:".
53-
54-
The current coverage report can be observed by running `./gradlew jacocoTestReport` and viewing `build/reports/jacoco/test/html/index.html`. This report is also produced as an artifact of the CircleCI build for the most recent Java version.
50+
Sometimes a gap in coverage is unavoidable, usually because the compiler requires us to provide a code path for some condition that in practice can't happen and can't be tested, or because of a known issue with the code coverage tool. In all such cases, please mark the code with an explanatory comment beginning with "COVERAGE:".

0 commit comments

Comments
 (0)