OpenZeppelin / OpenZeppelin/contracts-sui
[Feature]: Scope PR coverage to changed packages; move full-suite coverage to a nightly run
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Move
- Stars
- 49
- Forks
- 14
- PR merge metrics
- No merged PRs in 30d
Description
What is the feature you would like to see?
We currently run a full test suite coverage report on every PR (see test.yml). What's the value of measuring e.g. math/core coverage on a PR that only touches contracts/access?
We should split the coverage job into 2:
- Coverage only on packages "touched" by the PR runs on each PR -> measures the actual PR impact on a package's code coverage. Keeps current value, while making coverage job faster (thus cheaper $).
- Full test suite coverage -> runs nightly/weekly on
main, sanity check on the project-wide code coverage.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with .github/workflows/test.yml around the current coverage job at line 124. Trace how pull requests identify changed packages, then define the PR-scoped and scheduled full-suite coverage runs described in the issue. Done means PRs measure only touched packages while main receives a nightly or weekly full-suite sanity check.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions
- Domain
- ci-cd, testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100