Skip to content

Commit d81a14e

Browse files
authored
Merge pull request #157 from nhooyr/artifact
Upload coverage.html from test.sh as an artifact
2 parents 5dca20e + 68b99a9 commit d81a14e

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

.github/CONTRIBUTING.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,7 @@ CI must pass on your changes for them to be merged.
2929

3030
CI will ensure your code is formatted correctly, passes linting and tests.
3131
It will collect coverage and report it to [codecov](https://codecov.io/gh/nhooyr/websocket)
32-
and also upload a `out/coverage.html` artifact that you can click on to interactively
33-
browse coverage.
32+
and also upload a `coverage` artifact that you can download to inspect browse coverage.
3433

3534
You can run CI locally. The various steps are located in [ci/\*.sh](../ci).
3635

.github/workflows/ci.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,11 @@ jobs:
2222
- run: ./ci/test.sh
2323
env:
2424
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
25+
- name: Upload coverage.html
26+
uses: actions/upload-artifact@master
27+
with:
28+
name: coverage
29+
path: ci/out/coverage.html
2530
wasm:
2631
runs-on: ubuntu-latest
2732
container: docker://nhooyr/websocket-ci@sha256:b6331f8f64803c8b1bbd2a0ee9e2547317e0de2348bccd9c8dbcc1d88ff5747f

0 commit comments

Comments
 (0)