ChainSafe / ChainSafe/lodestar

Conditional status checks for PRs which do not require testing

Open
#6,686 5 comments 0 reactions 0 assignees View on GitHub
meta-feature-request scope-testing
Dominant language
TypeScript
Stars
1.4k
Forks
483
Avg merge
1d 18h
Merged PRs (30d)
156

Description

### Problem description

Currently, all PRs will go through all status checks. Whether it is an actual feature code change or just a documentation change, we should have a way to signal skipping checks for PRs that don't need to go through the entire testing process. Document change PRs don't really need to run unit/spec/sim tests.

### Solution description

We should create a label such as `status-skip-tests` we can use to signal the CI to skip certain status checks if tagged with this label.

### Additional context

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by locating the repository's CI workflow and status-check configuration, then trace how pull-request labels are exposed to jobs. Identify the unit, spec, and sim checks affected by the proposed status-skip-tests label, and verify that labeled documentation pull requests report the intended checks.

Written by the indexing model from the issue text.

Assessment

Domain
ci-cd
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.