Skip to content

Commit 30aee50

Browse files
authored
Merge pull request #13 from AeroRust/7-codecov
Enable Code Coverage Collection in GitHub Actions
2 parents 227d819 + 93bb49d commit 30aee50

File tree

1 file changed

+47
-48
lines changed

1 file changed

+47
-48
lines changed

.github/workflows/test.yml

Lines changed: 47 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -83,51 +83,50 @@ jobs:
8383
- name: Check proptest regressions
8484
run: |
8585
cargo test --all-features -- --include-ignored
86-
# Need to add token first see issue #7
87-
# coverage:
88-
# # use llvm-cov to build and collect coverage and outputs in a format that
89-
# # is compatible with codecov.io
90-
# #
91-
# # note that codecov as of v4 requires that CODECOV_TOKEN from
92-
# #
93-
# # https://app.codecov.io/gh/<user or org>/<project>/settings
94-
# #
95-
# # is set in two places on your repo:
96-
# #
97-
# # - https://github.com/jonhoo/guardian/settings/secrets/actions
98-
# # - https://github.com/jonhoo/guardian/settings/secrets/dependabot
99-
# #
100-
# # (the former is needed for codecov uploads to work with Dependabot PRs)
101-
# #
102-
# # PRs coming from forks of your repo will not have access to the token, but
103-
# # for those, codecov allows uploading coverage reports without a token.
104-
# # it's all a little weird and inconvenient. see
105-
# #
106-
# # https://github.com/codecov/feedback/issues/112
107-
# #
108-
# # for lots of more discussion
109-
# runs-on: ubuntu-latest
110-
# name: ubuntu / stable / coverage
111-
# steps:
112-
# - uses: actions/checkout@v4
113-
# with:
114-
# submodules: true
115-
# - name: Install stable
116-
# uses: dtolnay/rust-toolchain@stable
117-
# with:
118-
# components: llvm-tools-preview
119-
# - name: cargo install cargo-llvm-cov
120-
# uses: taiki-e/install-action@cargo-llvm-cov
121-
# - name: cargo generate-lockfile
122-
# if: hashFiles('Cargo.lock') == ''
123-
# run: cargo generate-lockfile
124-
# - name: cargo llvm-cov
125-
# run: cargo llvm-cov --locked --all-features --lcov --output-path lcov.info
126-
# - name: Record Rust version
127-
# run: echo "RUST=$(rustc --version)" >> "$GITHUB_ENV"
128-
# - name: Upload to codecov.io
129-
# uses: codecov/codecov-action@v4
130-
# with:
131-
# fail_ci_if_error: true
132-
# token: ${{ secrets.CODECOV_TOKEN }}
133-
# env_vars: OS,RUST
86+
coverage:
87+
# use llvm-cov to build and collect coverage and outputs in a format that
88+
# is compatible with codecov.io
89+
#
90+
# note that codecov as of v4 requires that CODECOV_TOKEN from
91+
#
92+
# https://app.codecov.io/gh/<user or org>/<project>/settings
93+
#
94+
# is set in two places on your repo:
95+
#
96+
# - https://github.com/jonhoo/guardian/settings/secrets/actions
97+
# - https://github.com/jonhoo/guardian/settings/secrets/dependabot
98+
#
99+
# (the former is needed for codecov uploads to work with Dependabot PRs)
100+
#
101+
# PRs coming from forks of your repo will not have access to the token, but
102+
# for those, codecov allows uploading coverage reports without a token.
103+
# it's all a little weird and inconvenient. see
104+
#
105+
# https://github.com/codecov/feedback/issues/112
106+
#
107+
# for lots of more discussion
108+
runs-on: ubuntu-latest
109+
name: ubuntu / stable / coverage
110+
steps:
111+
- uses: actions/checkout@v4
112+
with:
113+
submodules: true
114+
- name: Install stable
115+
uses: dtolnay/rust-toolchain@stable
116+
with:
117+
components: llvm-tools-preview
118+
- name: cargo install cargo-llvm-cov
119+
uses: taiki-e/install-action@cargo-llvm-cov
120+
- name: cargo generate-lockfile
121+
if: hashFiles('Cargo.lock') == ''
122+
run: cargo generate-lockfile
123+
- name: cargo llvm-cov
124+
run: cargo llvm-cov --locked --all-features --lcov --output-path lcov.info
125+
- name: Record Rust version
126+
run: echo "RUST=$(rustc --version)" >> "$GITHUB_ENV"
127+
- name: Upload to codecov.io
128+
uses: codecov/codecov-action@v4
129+
with:
130+
fail_ci_if_error: true
131+
token: ${{ secrets.CODECOV_TOKEN }}
132+
env_vars: OS,RUST

0 commit comments

Comments
 (0)