Skip to content

feat(coprocessor): post comment with coverage diff#2099

Merged
PanGan21 merged 5 commits intopanos/add-coverage-toolfrom
panos/post-coverage-diff
Mar 12, 2026
Merged

feat(coprocessor): post comment with coverage diff#2099
PanGan21 merged 5 commits intopanos/add-coverage-toolfrom
panos/post-coverage-diff

Conversation

@PanGan21
Copy link
Copy Markdown
Contributor

No description provided.

@cla-bot cla-bot bot added the cla-signed label Mar 11, 2026
@zama-ai zama-ai deleted a comment from github-actions bot Mar 11, 2026
@zama-ai zama-ai deleted a comment from github-actions bot Mar 11, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 11, 2026

Changed Lines Coverage

Coverage of added/modified lines: 100.0%

Per-file breakdown

Diff Coverage

Diff: origin/panos/add-coverage-tool...HEAD, staged and unstaged changes

No lines with coverage information in this diff.

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 11, 2026

🧪 CI Insights

Here's what we observed from your CI run for 0321864.

🟢 All jobs passed!

But CI Insights is watching 👀

@PanGan21 PanGan21 force-pushed the panos/post-coverage-diff branch 3 times, most recently from c475a4e to 0321864 Compare March 12, 2026 13:13
@PanGan21 PanGan21 marked this pull request as ready for review March 12, 2026 13:13
@PanGan21 PanGan21 requested a review from a team as a code owner March 12, 2026 13:13
@PanGan21 PanGan21 merged commit 3a3d25a into panos/add-coverage-tool Mar 12, 2026
103 checks passed
@PanGan21 PanGan21 deleted the panos/post-coverage-diff branch March 12, 2026 13:15
mergify bot pushed a commit that referenced this pull request Mar 13, 2026
…ows (#2086)

* feat(ci): add llvm-cov in ci

* feat(ci): set coverage threshold to 60

* fix(common): remove coverage thresholds, use opt level 1, apply coverage only on coprocessor

* fix(coprocessor): lint workflow

* fix(ci): capture cargo-llvm-cov report output by redirecting stderr

* fix(ci): fix warning

* fix(ci): handle failed generated report

* feat(coprocessor): post comment with coverage diff (#2099)

* feat(coprocessor): post comment with coverage diff

* fix(coprocessor): avoid template expansion

* fix(coprocessor): do not comment if workflow is cancelled

* feat(coprocessor): use sticky pr comment in workflow

* fix(coprocessor): unshallow fetch diff-cover from ci

* fix(coprocessor): use rust-toolchain-file action to read from rust-toolchain.toml

* fix(coprocessor): copy rust-toolchain.toml to root

* ci(coprocessor): handle empty-diff coverage and fork PR comment posting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants