Add merge-pr safety preflight skill#220
Open
andrei-hasna wants to merge 3 commits into
Open
Conversation
Contributor
|
All contributors have signed the CLA ✍️ ✅ |
andrei-hasna
force-pushed
the
feat/merge-pr-skill-mainline-20260712
branch
from
July 14, 2026 23:30
92ab649 to
cc4d33b
Compare
andrei-hasna
force-pushed
the
feat/merge-pr-skill-mainline-20260712
branch
from
July 15, 2026 08:11
cc4d33b to
cbae048
Compare
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.
Why
PR #215 contains the desired
merge-prskill work, but it targetsfleet/spark02-preserve-20260625-codewith. Retargeting that PR tomainwould include unrelated preservation-branch changes, which makes it unsuitable as a releaseable mainline PR.This PR supersedes #215 for releaseability by starting from current
mainand carrying only.codewith/skills/merge-pr/**fromde0db42c005d80b5d1ec900023311af53a645730.What changed
merge-prskill instructions and merge safety reference.merge_pr_preflight.pyhelper for PR mergeability snapshots.gh pr checksstatus fields such asbucket,state,status, andconclusion.Validation
PYTHONDONTWRITEBYTECODE=1 python3 .codewith/skills/merge-pr/tests/test_merge_pr_preflight.pypython3 -m py_compile .codewith/skills/merge-pr/scripts/merge_pr_preflight.pygit diff --checkgit diff --cached --checkshield reviewtodos redaction scan --fileover each staged filepython3 .codewith/skills/merge-pr/scripts/merge_pr_preflight.py 215 --repo hasna/codewith --mode preflightThe live PR #215 preflight was read-only and confirmed the current
gh pr checksJSON shape is parsed. It reported #215 asnot_mergeablebecause that preservation-branch PR currently has failing and pending checks.