Skip to content

Add badges - #134

Merged
jayvdb merged 2 commits into
mainfrom
readme
Sep 30, 2026
Merged

jayvdb merged 2 commits into
mainfrom
readme

Conversation

@jayvdb

@jayvdb jayvdb commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Documentation
    • Added README badges linking to the test workflow, coverage results, and the crate listing.
  • Bug Fixes
    • Added temporary exceptions for three security advisories affecting dependencies.
  • Chores
    • Updated test reporting and badge publishing for pushes to the main branch.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

Next included review available in 23 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 1e3f7edf-e3ca-4170-953b-0cb340801b9f

📥 Commits

Reviewing files that changed from the base of the PR and between b9821a3 and e8510c7.

📒 Files selected for processing (3)
  • .github/workflows/coverage.yaml
  • .mise/config.coverage.toml
  • config/nextest.toml

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 44eaa621-cdcb-4449-9730-176d6eebaa3f

📥 Commits

Reviewing files that changed from the base of the PR and between d35ff28 and b9821a3.

📒 Files selected for processing (7)
  • .github/workflows/coverage.yaml
  • .mise/config.coverage.toml
  • .mise/config.toml
  • README.md
  • config/deny.toml
  • config/osv-scanner.toml
  • config/semgrep/no-check-tool-mentions.yaml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The pull request adds Python JUnit reports and a test-count badge deployment flow, displays test, coverage, and crate badges in the README, and updates advisory ignore-list documentation and entries.

Changes

Coverage and test badges

Layer / File(s) Summary
Generate Python reports and test count
.mise/config.coverage.toml
The pytest-cov task writes a JUnit report for each Python module. The test-count-badge task counts Rust and Python test cases and writes the total to a JSON badge.
Upload results and deploy badge
.github/workflows/coverage.yaml
The workflow assigns permissions per job, uploads Python test results, and creates and uploads the badge artifact on pushes to main. A separate Pages job deploys the artifact.
Display repository badges
README.md, config/semgrep/no-check-tool-mentions.yaml
The README adds test, coverage, and crate badges. The Semgrep rule excludes the specified README Codecov references from its matches.

Advisory ignore configuration

Layer / File(s) Summary
Document and configure advisory ignores
.mise/config.toml, config/deny.toml, config/osv-scanner.toml
Comments describe how the cross-ecosystem ignore lists are used. Both scanner configurations add three Wasmtime advisory IDs with an expiry date of 2026-10-15.

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Feature

Sequence Diagram(s)

sequenceDiagram
  participant PytestCov as pytest-cov
  participant BadgeTask as test-count-badge
  participant CoverageWorkflow as coverage workflow
  participant Pages as GitHub Pages
  PytestCov->>BadgeTask: Python JUnit reports
  BadgeTask->>CoverageWorkflow: tests.json badge
  CoverageWorkflow->>Pages: Upload and deploy badge artifact
Loading

Merge Risk: ⚪ Minimal · up to b9821

The changes add test reporting and badge publication alongside consistent, time-limited advisory exceptions. No merge-blocking issue is established; merge after normal CI checks pass.

Architecture Summary

Architecture risk: 🔵 Low · up to b9821

The change affects 2 systems.

Changed systems: config, README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — config (service) was modified; 3 changed files map to changed impact.
  • observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in README.md: Added README badges for the test workflow, coverage, and et-ws-server crate, with links to their respective destinations.
  • observed — Modified behavior in config/deny.toml: Added comments explaining that the advisory ignore list covers multiple ecosystems, scanners consume their own slices, and out-of-scope or unused-ignore warnings are expected; entries should remain until their individual reasons indicate upstream has caught up.
  • observed — Modified behavior in config/deny.toml: Added Wasmtime advisory explanations and time-limited ignore entries for RUSTSEC-2026-0314, RUSTSEC-2026-0315, and RUSTSEC-2026-0316, each expiring 2026-10-15. The comments describe the affected guest behaviors, applicable runtime conditions, dependency constraint blocking the fixed version, and the stated follow-up when the expiry lapses.
  • observed — Modified behavior in config/osv-scanner.toml: Adds a comment that the list spans every language and unused-ignore warnings are expected.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly describes the README badge additions and related badge-generation workflow changes. It is concise and relevant to the primary change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@deepsource-io

deepsource-io Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in d35ff28...e8510c7 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
C# Sep 30, 2026 1:46a.m. Review ↗
C & C++ Sep 30, 2026 1:46a.m. Review ↗
Docker Sep 30, 2026 1:46a.m. Review ↗
Java Sep 30, 2026 1:46a.m. Review ↗
JavaScript Sep 30, 2026 1:46a.m. Review ↗
Python Sep 30, 2026 1:46a.m. Review ↗
Rust Sep 30, 2026 1:46a.m. Review ↗
Secrets Sep 30, 2026 1:46a.m. Review ↗
Code coverage Sep 30, 2026 2:33a.m. Review ↗

Code Coverage Summary

Language Line Coverage (Overall)
Aggregate
70.8%
Python
89.6%
Rust
69.2%

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@jayvdb
jayvdb marked this pull request as ready for review September 30, 2026 01:10
@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@jayvdb
jayvdb merged commit 42aba28 into main Sep 30, 2026
44 of 45 checks passed
@jayvdb
jayvdb deleted the readme branch September 30, 2026 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants