Skip to main content

merge-pr

Squash-merge a cube-js/cube pull request with a clean commit message body. Use whenever you are about to merge a PR — `gh pr merge`, "merge it", "ship it", "land this PR" — so the squash commit body gets a written summary instead of the raw PR description.

소스 정보

저장소
cube-js/cube
최근 소스 활동
2026년 9월 24일 14:37
감지된 SKILL.md 언어
영어
스타
21,146
포크
2,179

설치 방법

기본적으로 소스를 먼저 확인하는 Prompt가 선택됩니다. 직접 명령으로 전환하거나 로컬 사본을 다운로드할 수도 있습니다.

소스 파일 검토

설치 여부를 결정하기 전에 SKILL.md와 SkillsMP에 표시된 보조 파일을 읽어 보세요.

SKILL.md 표시 중

SKILL.md
소스 지침 · 읽기 전용 미리보기
name
merge-pr
description
Squash-merge a cube-js/cube pull request with a clean commit message body. Use whenever you are about to merge a PR — `gh pr merge`, "merge it", "ship it", "land this PR" — so the squash commit body gets a written summary instead of the raw PR description.
argument-hint
[<pr-number>]
# Merge a PR Master only allows squash merges, and the repo default squash message is the PR title plus the PR description. Without `--body`, the raw description — template checklist, HTML comments, review chatter — lands in master history forever. Always pass `--body`. Do not pass `--subject`: the commit title is the PR title. If the PR title itself is wrong, ask the author to fix it on the PR instead of overriding it at merge time. ## 1. Check the PR is mergeable ```bash gh pr view <n> --json number,title,body,state,isDraft,mergeable,commits,headRefName ``` Stop and report back if the PR is a draft or has conflicts. Do not use `--admin` unless the user explicitly asked for it. ## 2. Write the body Describe the final state of the change, not how the branch got there: - 1–15 lines (not counting benchmark tables and CVE lists below): what changed and why. Take it from the PR description, not from the commit list. Omit the PR template checklist and HTML comments. - Benchmark / measurement results from the PR are welcome — keep them, as a markdown table (`| case | before | after |`) with the setup in one line. - Fixed CVEs / security advisories (typical for dependency bumps) are welcome — list them one per line with the affected package, e.g. `CVE-2025-12345 (qs)`, `GHSA-xxxx-xxxx-xxxx (undici)`. - Linked issues / tickets: `Fixes #123`, `CUB-1234`. - `BREAKING CHANGE: ...` footer if applicable. - Trailers last: one `Co-authored-by:` per distinct co-author across the PR's commits (deduplicated, human authors other than the PR author, plus the AI attribution line if any commit carried one). An empty body is fine for a trivial PR whose title says it all — pass `--body ""` explicitly, never omit the flag. Never paste the per-commit messages. ## 3. Merge ```bash gh pr merge <n> --squash --body "$(cat <<'EOF' The __MAX_SOURCE_ROW_LIMIT sentinel was parsed as usize and silently dropped, so queries ran without any LIMIT. Fixes #12300 Co-authored-by: Jane Doe <jane@example.com> EOF )" ``` Then confirm with `gh pr view <n> --json mergeCommit --jq .mergeCommit.oid` and `git fetch && git show -s --format=%B <oid>` (not `git log -1 origin/master` — another merge may have landed since), and report the merged commit to the user.
GitHub에서 보기