-
Type:
Improvement
-
Resolution: Unresolved
-
Priority:
Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
None
-
Storage Engines, Storage Engines - Foundations
-
SE Foundations - 2025-11-07
-
2
Description
Currently, the evergreen runs the full test suite on a pull request even if the code formatting check (s_all) fails. This leads to wasted resources and time, as a PR with formatting issues cannot be merged anyway.
For example, when a developer forgets to run s_all locally, they have to push a new commit to fix the formatting, which then triggers the entire test suite to run again.
Proposed Solution
Modify the evergreen PR testing pipeline to make the s_all test a prerequisite. The evergreen should stop and fail immediately if s_all fails. This would prevent the more resource-intensive tests from running until all code formatting is corrected.
Acceptance Criteria
- The evergreen PR pipeline checks for s_all test status first.
- If the s_all test fails, the evergreen terminates without running subsequent tests.