Skip to content

Commit 6a6dc4b

Browse files
docs(testing): coverage guide should not prescribe a removed uploader (#753)
Completes the Codecov removal in this repo. `wiki/testing/guide.md` had a copy-paste block telling contributors to add `uses: codecov/codecov-action@v3` with a `coverage.lcov` file — Codecov is not installed here or on the account, so following the guide would produce the same red CI this sweep removed, and Dependabot would eventually re-pin the tag. Rewritten to state what `.github/workflows/ci.yml` really does (visibility-only coverage via `bisect-ppx-report` + `actions/upload-artifact`) and to point at `docs/standards/TESTING.adoc`, so the doc and the CI can't drift again. No functional workflow change: this file is documentation only.
1 parent 3d8d3d8 commit 6a6dc4b

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

wiki/testing/guide.md

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -609,15 +609,21 @@ aspm coverage --format lcov --output coverage.lcov
609609

610610
### Coverage in CI
611611

612+
Coverage is visibility-only: the report is attached to the run as an artifact, not shipped
613+
to a third-party service. This is what `.github/workflows/ci.yml` actually does (see
614+
`docs/standards/TESTING.adoc`, "Coverage (visibility-only)") — no upload token, no external
615+
account, and nothing for Dependabot to keep bumping.
616+
612617
```yaml
613618
# .github/workflows/ci.yml
614619
- name: Run tests with coverage
615620
run: aspm test --coverage
616621

617-
- name: Upload coverage
618-
uses: codecov/codecov-action@v3
622+
- name: Upload coverage report
623+
uses: actions/upload-artifact@v7.0.1
619624
with:
620-
file: coverage.lcov
625+
name: coverage-lcov
626+
path: coverage.lcov
621627
```
622628
623629
---

0 commit comments

Comments
 (0)