Document the required CI checks and the job-rename hazard - #300
Draft
CarsonDavis wants to merge 1 commit into
Draft
Document the required CI checks and the job-rename hazard#300CarsonDavis wants to merge 1 commit into
CarsonDavis wants to merge 1 commit into
Conversation
7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Documents the merge-gating rules that #201 adds to the
require-prruleset, in the place a contributor will look before touching workflows.development/main/productionmust passunit,config-generation,e2e (full), ande2e (lean)(all fromplaywright-tests.yml) on top of the existing 1-approval requirement; "not green" includes still-running and never-reported checks.playwright-tests.ymlsilently blocks every PR until an admin updates the ruleset — job renames there are ruleset-affecting changes.The companion ruleset change (adding the
required_status_checksrule to rulesetrequire-pr, id 12749875) is a live-repo admin action applied separately once this PR is reviewed; the payload has been verified against the live config and the check contexts on the latest merged PR head.Closes #201