Skip to content

docs: CONTRIBUTING의 node/pnpm 버전 실제값으로 수정#227

Merged
yujeong-jeon merged 1 commit into
mainfrom
docs/fix-contributing-versions
Jun 30, 2026
Merged

docs: CONTRIBUTING의 node/pnpm 버전 실제값으로 수정#227
yujeong-jeon merged 1 commit into
mainfrom
docs/fix-contributing-versions

Conversation

@yujeong-jeon

Copy link
Copy Markdown
Contributor

무엇을

CONTRIBUTING.md시스템 요구 사항 버전이 실제 레포 설정과 달라, 신입 기여자가 문서대로 환경을 세팅하면 CI와 어긋나는 문제를 수정합니다.

항목 기존(문서) 변경(실제값) 근거
node 18.12.0 22.11.0 .nvmrc (CI도 Node 22 사용)
pnpm 8.6.6 10.5.2 package.jsonpackageManager

신입 온보딩 중 발견했습니다. 문서의 node@18 / pnpm@8.6.6을 그대로 설치하면 pnpm install --frozen-lockfile이나 빌드 단계에서 CI(Node 22 / pnpm 10.5.2)와 동작이 달라질 수 있어, 진입 장벽이자 디버깅 비용으로 이어집니다.

범위

  • 문서 2줄만 수정, 코드/패키지 변경 없음 → changeset 불필요
  • 패키지 영향 없음

🤖 Generated with Claude Code

CONTRIBUTING.md의 시스템 요구 사항이 실제와 달라 신입 기여자가 그대로
따라 하면 CI와 어긋나는 환경으로 세팅되는 문제를 수정.

- node: 18.12.0 -> 22.11.0 (.nvmrc 기준)
- pnpm: 8.6.6 -> 10.5.2 (package.json packageManager 기준)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@yujeong-jeon yujeong-jeon requested a review from a team as a code owner June 29, 2026 13:02
@npayfebot

Copy link
Copy Markdown
Collaborator

🔍 No files have been changed

Latest commit: 5d45f21

Please check your commit.

If packages_dir is not specified or only markdown files are in the changes, detection may fail.

powered by: naverpay changeset detect-add actions

@npayfebot

Copy link
Copy Markdown
Collaborator

🔍 No files have been changed

Latest commit: 5d45f21

Please check your commit.

powered by: naverpay size-action

@myamaak myamaak left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정 감사합니다!

@yujeong-jeon yujeong-jeon merged commit 11f340c into main Jun 30, 2026
7 checks passed
@yujeong-jeon yujeong-jeon deleted the docs/fix-contributing-versions branch June 30, 2026 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants