Conditional status checks for PRs which do not require testing about lodestar HOT 5 OPEN

Maybe we can do it based on modified files, e.g. if only edits in docs/** then just run linting, spellcheck, .. and no test suites

from lodestar.

Is there anything else we foresee not requiring status check tests to run?

from lodestar.

I personally would love to see spec-tests being skippable for unrelated PRs

Yeah me too. There are some PRs that shouldn't need to go through these when they obviously don't modify for the functionality of Lodestar in any way.

Currently, for unstable merge, the tests that must pass are:

• Spec tests (20)
• Unit Tests (20)
• Lint (20)
  (Note, it shouldn't even be this minimal, but due to ongoing work for other tests, this is currently the setting to bypass inconsistencies)

All tests for PRs:

• Benchmark / run (pull_request)
• CodeQL / Analyze (javascript) (pull_request)
• Check docs / Docs spellcheck (pull_request)
• Lint PR title / Validate PR title (pull_request_target)
• Sim merge execution/builder tests / Sim merge tests (pull_request) S
• Sim tests / Build (pull_request)
• Tests / Build (20) (pull_request)
• Tests / Lint (20) (pull_request)
• Sim tests / Multifork sim test (pull_request)
• Sim tests / Endpoint sim tests (pull_request)
• Tests / Type Checks (20) (pull_request)
• Sim tests / Deneb sim tests (pull_request)
• Tests / E2E Tests (20) (pull_request)
• Sim tests / Eth backup provider sim tests
• Tests / Browser Tests (20) (pull_request)
• Sim tests / Mixed clients sim tests (pull_request)
• Tests / Spec tests (20) (pull_request)
• Tests / Unit Tests (20) (pull_request)
• license/cla

For things that don't require extensive testing (docs changes, some repo maintenance chores, etc.), we should just have it run:

• Check docs / Docs spellcheck (pull_request)
• Lint PR title / Validate PR title (pull_request_target)
• license/cla

Maybe we can do it based on modified files, e.g. if only edits in docs/** then just run linting, spellcheck, .. and no test suites

There are some things that don't necessarily touch only docs/**. For example, some website maintenance to our docs are labeled as chore: and perhaps those could be excluded too? This is why I thought of the label method as a conditional for the CI to skip certain tests and can be appended by the PR author based on their judgement and the reviewers.

from lodestar.