Discussion: Consider renaming /run-skipped-ci to /run-full-ci #2001
Description
Background
Currently, the command to trigger the full CI suite (including minimum dependency tests) is /run-skipped-ci.
Question
Should we rename this to /run-full-ci for better clarity?
Current name: /run-skipped-ci
Pros:
- Technically accurate - you're running tests that are "skipped" on regular PRs
- Emphasizes that you're running additional tests beyond the default
Cons:
- Requires understanding the negative framing ("skipped")
- Less intuitive for new contributors
- Could be confused with "run only the skipped tests"
Alternative: /run-full-ci
Pros:
- More intuitive - clearly states you're running the full test suite
- Positive framing ("full" vs "skipped")
- Easier to understand for new contributors
Cons:
- Less technically precise
- Doesn't emphasize that PRs normally skip some tests
Additional Context
The command was recently implemented in PR #1995 to address CI reliability issues. Usage is documented in CONTRIBUTING.md.
Decision Needed
- Keep
/run-skipped-ci- no change needed - Rename to
/run-full-ci- requires updating workflow, docs, and potentially closing/reopening the command in GitHub - Support both names (alias)
Priority
Low - This is purely about user experience and documentation clarity.
Related
- PR Improve Streamed Components retry logic #1995: CI comment trigger security and reliability improvements
CONTRIBUTING.md: Command documentation