chore(github): PR 템플릿 개편 및 /pr 커맨드 추가 - #35
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
범위 체크리스트에 apps/react-zpl만 있어 별도 npm 패키지인 zpl-core와 zpl-viewer를 고를 수 없었다. 제목 규칙도 적혀 있지 않아 squash 제목이 그대로 릴리즈 노트 한 줄이 된다는 사정이 드러나지 않았다. - 범위를 실제 워크스페이스(zpl-core·react-zpl·zpl-viewer·demos·docs· tests·CI)로 교체하고 배포 대상임을 함께 표기 - 제목 규칙 안내 추가 — squash 제목이 main 커밋이자 릴리즈 노트가 되는 사정 - "의도적으로 안 한 것 / 후속" 섹션 추가 — 누락과 의도를 구분 - 검증을 체크박스가 아니라 값으로 적게 변경 (`pnpm verify` 결과) - 공개 패키지 영향(공개 API·ZPL 출력 변화·태그 필요 여부) 섹션 추가 - 리뷰 포인트/위험 지점 섹션 추가 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
PR 템플릿은 본문을 제공해 PR을 만들면 아예 거치지 않는다. 실제로 머지된 PR 어디에도 템플릿 섹션이 채워진 흔적이 없다. 템플릿을 읽어 채우는 경로가 있어야 실효가 생긴다. - 본문 구조의 단일 출처는 `.github/pull_request_template.md`. 커맨드는 규칙을 다시 정의하지 않고 채우기만 한다 - 검증은 `pnpm verify` 결과를 값으로 적게 한다 - 공개 패키지 영향은 생성되는 ZPL 문자열 변화를 판정 기준으로 둔다 - push·PR 생성·라벨 부여는 승인 후에만 실행한다 - 라벨은 레포에 정의된 것 중에서 고르고, 애매하면 사용자에게 묻는다 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Bori-github
force-pushed
the
docs/pr-template
branch
from
July 28, 2026 09:00
63ff921 to
e44ef84
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.
요약 / 의도
PR 템플릿의 범위 목록이 실제 워크스페이스와 어긋나 있었고(별도 npm 패키지인
zpl-core·zpl-viewer를 고를 수 없었다), 검증 결과를 남길 자리가 없었다.
템플릿을 실제 구조에 맞추고, 그것을 읽어 채우는
/pr커맨드를 함께 넣는다.범위
docs항목은 문서 사이트(Rspress)를 뜻하므로 체크하지 않았다 — 이 PR이건드리는 건
.github·.claude메타 파일이다.이번에 한 것
pnpm verify결과).claude/commands/pr.md— 템플릿을 읽어 채우고 승인 후에만 push·생성의도적으로 안 한 것 / 후속
없다. CLAUDE.md 정비 PR에서 함께 다루는 게 순서다
.claude/settings.json(공유 권한)·훅은 별도 PR공개 패키지 영향
해당 없음 —
apps/아래 변경이 없다. 공개 API·생성 ZPL·버전 모두 무관.설계 · 결정
삽입되지 않는다. 실제로 머지된 PR 어디에도 템플릿 섹션이 채워진 흔적이 없다
(docs(packages): npm README 추가 및 문서 도메인 오타 수정 #26·fix(ci): provenance를 NPM_CONFIG_PROVENANCE env로 켜기 #31·chore(demo-electron): 패키지명을 @zpl-kit/demo-electron으로 바꾸고 private 지정 #33 모두 자유 형식). 템플릿을 읽어 채우는 경로가 있어야 실효가
생겨서 커맨드를 같은 PR에 넣었다
커맨드는 채우기만 한다. 두 곳에 규칙이 흩어지면 곧 어긋난다
검증 (체크가 아니라 값으로)
pnpm verify: 통과 (exit 0 · oxlint 125 files · prettier 통과 · 42 tests)verify·test·test:watch·build:libs)와경로 6개가 모두 실재함을 확인
관찰된 적이 없다(모든 PR이 본문을 제공해 생성됐다). 경로는 GitHub 표준이라
문제 소지는 낮다
리뷰 포인트 / 위험 지점
breaking change 판정 기준으로 못 박았다. 이 기준이 맞는지 봐주면 좋겠다
/pr커맨드로 작성됐다 — 커맨드의 첫 실전 적용이다