Skip to content

fix: Node.js 버전 관리 통일#49

Merged
mgYang53 merged 2 commits intomainfrom
fix/node-version-48
Feb 1, 2026
Merged

fix: Node.js 버전 관리 통일#49
mgYang53 merged 2 commits intomainfrom
fix/node-version-48

Conversation

@mgYang53
Copy link
Contributor

@mgYang53 mgYang53 commented Feb 1, 2026

🚀 풀 리퀘스트 제안

📋 작업 내용

CI/CD 워크플로우와 로컬 개발 환경의 Node.js 버전 관리가 일관성 없이 설정되어 있어 이를 .nvmrc 파일을 단일 소스로 통합했습니다.

🔧 변경 사항

  • .nvmrc: 2020.20.0 (Node 20 LTS 버전으로 고정)
  • package.json: engines >=20.19 <21>=20.20.0 <21
  • .github/workflows/ci.yml: node-version: '20'node-version-file: '.nvmrc'
  • .github/workflows/deploy.yml: node-version: '20'node-version-file: '.nvmrc'

📄 기타

  • 이제 Node 버전 변경 시 .nvmrc 파일만 수정하면 CI/CD에도 자동 반영됩니다.
  • Node 20.20.0은 현재 Node 20 버전대의 최신 LTS입니다.

closes #48

Summary by CodeRabbit

릴리스 노트

  • Chores

    • Node.js 버전 요구사항을 20.20.0으로 업데이트했습니다.
    • CI/CD 워크플로우를 개선하여 Node.js 버전을 중앙집중식 구성 파일에서 관리하도록 변경했습니다.
  • Documentation

    • 개발 가이드 문서가 제거되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

- .nvmrc를 20.20.0 (LTS)으로 고정
- package.json engines를 >=20.20.0 <21로 업데이트
- CI/CD workflow에서 node-version-file로 .nvmrc 참조
@mgYang53 mgYang53 linked an issue Feb 1, 2026 that may be closed by this pull request
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Walkthrough

Node.js 버전 관리를 통일합니다. CI/CD 워크플로우를 .nvmrc 파일 참조로 변경하고, .nvmrc를 20.20.0으로 명시하며, package.json engines도 >=20.20.0으로 업데이트합니다. 개발 문서는 제거됩니다.

Changes

Cohort / File(s) Summary
CI/CD 워크플로우
.github/workflows/ci.yml, .github/workflows/deploy.yml
node-version: '20'node-version-file: '.nvmrc'로 변경하여 .nvmrc를 단일 소스로 관리.
버전 명시
.nvmrc, package.json
.nvmrc를 20에서 20.20.0으로, package.json engines를 >=20.19 <21에서 >=20.20.0 <21로 업데이트.
문서
docs/DEVELOPMENT.md
노드 설치, Corepack, pnpm 설정 관련 개발 가이드 문서 삭제.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • choiyoungae
  • haruyam15
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 주요 변경사항(Node.js 버전 관리 통일)을 명확하게 설명하고 있어 변경사항과 충분히 연관되어 있습니다.
Linked Issues check ✅ Passed 모든 코드 변경사항이 #48 이슈의 요구사항을 충족합니다: .nvmrc를 20.20.0으로 고정, package.json engines 업데이트, CI/CD 워크플로우 수정.
Out of Scope Changes check ✅ Passed docs/DEVELOPMENT.md 삭제는 불필요한 문서 제거로 범위 내 변경사항입니다. 모든 변경이 Node.js 버전 관리 통일 목표와 관련있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/node-version-48

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mgYang53 mgYang53 changed the title fix: unify Node.js version management fix: Node.js 버전 관리 통일 Feb 1, 2026
@mgYang53 mgYang53 self-assigned this Feb 1, 2026
@mgYang53 mgYang53 added chore 설정과 기타 작업들을 포함한 추가 및 수정 fix 기존 기능 수정 labels Feb 1, 2026
@mgYang53 mgYang53 merged commit f887183 into main Feb 1, 2026
2 checks passed
@mgYang53 mgYang53 deleted the fix/node-version-48 branch February 1, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 설정과 기타 작업들을 포함한 추가 및 수정 fix 기존 기능 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] Node.js 버전 불일치 수정

2 participants