Skip to content

ci: include coverage from running build scripts#6007

Draft
davidhewitt wants to merge 11 commits intoPyO3:mainfrom
davidhewitt:build-script-coverage
Draft

ci: include coverage from running build scripts#6007
davidhewitt wants to merge 11 commits intoPyO3:mainfrom
davidhewitt:build-script-coverage

Conversation

@davidhewitt
Copy link
Copy Markdown
Member

This attempts to improve coverage measurement of pyo3-build-config by measuring the coverage of invocation of build.rs files.

I refactored a few common pieces about coverage measurement into helper actions so that I could re-use them from test-version-limits job.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 30, 2026

Merging this PR will not alter performance

✅ 125 untouched benchmarks


Comparing davidhewitt:build-script-coverage (818ee2c) with main (65d6a36)

Open in CodSpeed

@ngoldbaum
Copy link
Copy Markdown
Contributor

Thanks for working on this! I noticed this looking at our clverage output for my build configuration PR. Happy to help out if you want a hand or an eye for review.

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