Skip to content

test: add code coverage reporting#275

Merged
ajcasagrande merged 22 commits into
mainfrom
dbermudez/aip-326-add-code-coverage-reporting
Sep 20, 2025
Merged

test: add code coverage reporting#275
ajcasagrande merged 22 commits into
mainfrom
dbermudez/aip-326-add-code-coverage-reporting

Conversation

@debermudez

@debermudez debermudez commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Add codecov integration: https://app.codecov.io/github/ai-dynamo/aiperf
Remove unused documentation script as clean up as well.

@linear

linear Bot commented Sep 17, 2025

Copy link
Copy Markdown
AIP-326 Add code coverage reporting

Add a github workflow action to generate a coverage report

@github-actions github-actions Bot added the feat label Sep 17, 2025
@debermudez debermudez changed the title feat: add code coverage reporting test: add code coverage reporting Sep 17, 2025
@github-actions github-actions Bot added test and removed feat labels Sep 17, 2025
@debermudez debermudez force-pushed the dbermudez/aip-326-add-code-coverage-reporting branch 2 times, most recently from 0fca9eb to c3018ad Compare September 18, 2025 21:50
@codecov

codecov Bot commented Sep 18, 2025

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.

ℹ️ 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 ☂️

@the-david-oy the-david-oy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Excellent work!

What do you think of adding this to the unit tests action? That way pytest isn't running twice. It looks like this is similar to the command we have under make coverage, so you could probably update that one and then add the upload step to that action.

@debermudez debermudez force-pushed the dbermudez/aip-326-add-code-coverage-reporting branch from 92d0592 to 05a296c Compare September 19, 2025 23:06

@ajcasagrande ajcasagrande left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice!

@ajcasagrande ajcasagrande merged commit 15a6672 into main Sep 20, 2025
4 checks passed
@ajcasagrande ajcasagrande deleted the dbermudez/aip-326-add-code-coverage-reporting branch September 20, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants