Skip to content

Document stacked dependent PR handling in split PR skill#5496

Open
wujingyue wants to merge 1 commit into
NVIDIA:mainfrom
wujingyue:codex/clean-split-pr-dependency-guidance
Open

Document stacked dependent PR handling in split PR skill#5496
wujingyue wants to merge 1 commit into
NVIDIA:mainfrom
wujingyue:codex/clean-split-pr-dependency-guidance

Conversation

@wujingyue

@wujingyue wujingyue commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add explicit split-PR guidance for stacked dependent PRs: use GitHub base/diffbase pull-request/<base PR number> instead of the base PR author branch.
  • Document the follow-up rule to retarget and refresh dependent PRs back to main before the base PR merges, avoiding GitHub auto-closing stacked PRs and losing review discussion.

Validation

  • python3 /home/me/.codex/skills/.system/skill-creator/scripts/quick_validate.py skills/mcore-split-pr
  • Expected existing failure: argument, user_invocable, and when_to_use are still unsupported frontmatter keys.

Signed-off-by: Jingyue Wu <wujingyue@gmail.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@wujingyue wujingyue requested review from Phlip79 and ko3n1g June 25, 2026 04:52
@wujingyue wujingyue changed the title [codex] Clean up split PR dependency guidance Clean up split PR dependency guidance Jun 25, 2026
@wujingyue wujingyue changed the title Clean up split PR dependency guidance Document stacked dependent PR handling in split PR skill Jun 25, 2026
@wujingyue wujingyue marked this pull request as ready for review June 25, 2026 04:56
@svcnvidia-nemo-ci svcnvidia-nemo-ci added the docs-only documentation only (docs or docstrings) label Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-only documentation only (docs or docstrings)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants