Skip to content

chore(github): PR 템플릿 개편 및 /pr 커맨드 추가 - #35

Merged
Bori-github merged 2 commits into
mainfrom
docs/pr-template
Jul 28, 2026
Merged

chore(github): PR 템플릿 개편 및 /pr 커맨드 추가#35
Bori-github merged 2 commits into
mainfrom
docs/pr-template

Conversation

@Bori-github

@Bori-github Bori-github commented Jul 28, 2026

Copy link
Copy Markdown
Owner

요약 / 의도

PR 템플릿의 범위 목록이 실제 워크스페이스와 어긋나 있었고(별도 npm 패키지인
zpl-core·zpl-viewer를 고를 수 없었다), 검증 결과를 남길 자리가 없었다.
템플릿을 실제 구조에 맞추고, 그것을 읽어 채우는 /pr 커맨드를 함께 넣는다.

범위

  • CI · 툴링 · 설정

docs 항목은 문서 사이트(Rspress)를 뜻하므로 체크하지 않았다 — 이 PR이
건드리는 건 .github·.claude 메타 파일이다.

이번에 한 것

  • 범위 목록을 실제 워크스페이스로 교체하고 배포 대상임을 함께 표기
  • 검증을 체크박스가 아니라 값으로 적게 변경 (pnpm verify 결과)
  • "의도적으로 안 한 것 / 후속", "공개 패키지 영향", "리뷰 포인트" 섹션 추가
  • .claude/commands/pr.md — 템플릿을 읽어 채우고 승인 후에만 push·생성

의도적으로 안 한 것 / 후속

  • 커밋 커맨드는 함께 넣지 않았다 — 커밋 컨벤션 문서도 commitlint도 이 레포엔
    없다. CLAUDE.md 정비 PR에서 함께 다루는 게 순서다
  • .claude/settings.json(공유 권한)·훅은 별도 PR

공개 패키지 영향

해당 없음 — apps/ 아래 변경이 없다. 공개 API·생성 ZPL·버전 모두 무관.

설계 · 결정

검증 (체크가 아니라 값으로)

  • pnpm verify: 통과 (exit 0 · oxlint 125 files · prettier 통과 · 42 tests)
  • 템플릿이 참조하는 스크립트 4개(verify·test·test:watch·build:libs)와
    경로 6개가 모두 실재함을 확인
  • 새 동작에 테스트 추가: 해당없음 — 마크다운 문서·커맨드 정의
  • 자동 검증 불가: 템플릿이 PR 작성 화면에 실제로 삽입되는지는 이 레포에서 한 번도
    관찰된 적이 없다(모든 PR이 본문을 제공해 생성됐다). 경로는 GitHub 표준이라
    문제 소지는 낮다

리뷰 포인트 / 위험 지점

  • 커맨드 6번(공개 패키지 영향)에서 "생성되는 ZPL 문자열이 달라지는가"를
    breaking change 판정 기준
    으로 못 박았다. 이 기준이 맞는지 봐주면 좋겠다
  • 이 PR 자체가 /pr 커맨드로 작성됐다 — 커맨드의 첫 실전 적용이다

@Bori-github Bori-github self-assigned this Jul 28, 2026
@Bori-github Bori-github added the chore Maintenance, tooling, or CI — no user-facing change label Jul 28, 2026
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
zpl-viewer Ignored Ignored Preview Jul 28, 2026 9:00am

Bori-github and others added 2 commits July 28, 2026 18:00
범위 체크리스트에 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
Bori-github merged commit e4d9424 into main Jul 28, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance, tooling, or CI — no user-facing change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant