Skip to content

Code and doc coverage w/ Codecov#325

Merged
JarettForzano merged 7 commits intodevfrom
jf/code-and-doc-coverage
May 5, 2026
Merged

Code and doc coverage w/ Codecov#325
JarettForzano merged 7 commits intodevfrom
jf/code-and-doc-coverage

Conversation

@JarettForzano
Copy link
Copy Markdown
Collaborator

@JarettForzano JarettForzano commented May 5, 2026

Summary

Used to create badges that will live inside of the repository. The purpose is to provide metrics that we can use to judge how tested and documented this project is. In the future we can enforce certain metric thresh-holds that fail if we are doing a bad job.
image

Changes get populated here https://app.codecov.io/gh/vals-ai/Valkyrie/tree/jf%2Fcode-and-doc-coverage. This will be public to everyone

Changes

  • Added cli-unit-test workflow
  • Added docstr-coverage workflow
  • Updated tracker unit test workflow to include coverage
  • Added badges to the REDME.md file and updated pyproject to include new dependencies
  • Enabled github pages to host json with docstring coverage included

Related Issues

Closes #

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactor
  • Docs / config

Testing

  • Added/updated unit tests
  • Manually tested

Checklist

  • Self-reviewed the diff
  • No debug/dead code left in
  • Docs updated if needed

@assert-app
Copy link
Copy Markdown

assert-app Bot commented May 5, 2026

Review on Assert →

@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@JarettForzano JarettForzano changed the title [WIP]: Code and doc coverage w/ Codecov Code and doc coverage w/ Codecov May 5, 2026
@JarettForzano JarettForzano marked this pull request as ready for review May 5, 2026 17:20
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@JarettForzano JarettForzano merged commit e20d495 into dev May 5, 2026
8 of 9 checks passed
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