Skip to content

Initial PR with codecov report logging.#2581

Draft
arav-agarwal2 wants to merge 7 commits into
masterfrom
arav-codecov-impl
Draft

Initial PR with codecov report logging.#2581
arav-agarwal2 wants to merge 7 commits into
masterfrom
arav-codecov-impl

Conversation

@arav-agarwal2
Copy link
Copy Markdown
Contributor

@arav-agarwal2 arav-agarwal2 commented Apr 22, 2026

This is a fpreliminary codecov report addition.

I've added a basic tests file, used Claude to generate some basic tests for the submission checker, and gone from there.

The report can be seen here: https://app.codecov.io/github/mlcommons/inference/tree/arav-codecov-impl

Notes:

  • I still need to add the mlcr integration tests to this if possible, as currently this doesn't utilize them and only uses pytest.
  • The system currently only looks at PRs to this branch, not master. I want to make sure buy-in exists for adding tests before I start trying to figure out if I should include more, and I need to confirm my testing completeness is correct.

@github-actions
Copy link
Copy Markdown
Contributor

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@codecov-commenter
Copy link
Copy Markdown

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.

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

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