Skip to content

feat: PR 관련 CI workflow 추가#45

Merged
mgYang53 merged 3 commits intomainfrom
feat/add-pr-ci-42
Feb 1, 2026
Merged

feat: PR 관련 CI workflow 추가#45
mgYang53 merged 3 commits intomainfrom
feat/add-pr-ci-42

Conversation

@mgYang53
Copy link
Contributor

@mgYang53 mgYang53 commented Feb 1, 2026

Summary

  • PR 생성/업데이트 시 코드 품질 검사를 수행하는 CI 워크플로우 추가
  • lint, format, build 검사 자동화
  • concurrency 설정으로 중복 실행 방지

Test plan

  • PR 생성 시 CI 워크플로우가 자동 실행되는지 확인
  • lint, format, build 각 단계가 정상 동작하는지 확인

closes #42

Summary by CodeRabbit

릴리스 노트

  • Chores
    • CI 파이프라인이 추가되어 PR에서 코드 검사·빌드가 자동 실행됩니다.
    • 배포 스크립트의 경로 표기 일부가 정리되었습니다.
  • 버그 수정
    • 온보딩 화면의 버튼 마크업 문제를 수정하여 버튼 레이블이 정상적으로 표시됩니다.
  • 스타일
    • 여러 UI 및 API 관련 파일의 코드 포맷팅 및 마크업 정리가 이루어졌습니다.

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

PR 생성/업데이트 시 lint, format, build 검사 수행
@mgYang53 mgYang53 linked an issue Feb 1, 2026 that may be closed by this pull request
7 tasks
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Walkthrough

PR에 CI 워크플로우 .github/workflows/ci.yml을 신규 추가하고, 배포 워크플로우의 문자열/경로 스타일과 여러 소스 파일에서 사소한 문법·포맷 수정을 적용했습니다. 변경은 주로 CI 추가 및 코드 포맷/구문 정정에 국한됩니다. (≤50단어)

Changes

Cohort / File(s) Summary
CI Workflow
.github/workflows/ci.yml
PR(pull_request on main, dev) 트리거로 pnpm(10.28.0)·Node.js 20 환경을 설정하고 lint, format:check, type check + build 단계를 실행하는 GitHub Actions 워크플로우 신규 추가.
Deploy workflow
.github/workflows/deploy.yml
SCP 경로 문자열 인용 스타일 변경("dist/*"'dist/*') 및 원격 정리 경로 오타(대시 관련) 수정. 제어 흐름 불변.
API / retry / endpoints
src/api/interceptors.ts, src/api/retry.ts, src/features/gatherings/gatherings.endpoints.ts
axios 인터셉터 오류 핸들러 구문 정리(쉼표→중괄호), setupRetryInterceptor 기본 파라미터 공백 포맷 수정, JOIN_REQUEST 엔드포인트 포맷팅 변경 — 동작 불변.
User API
src/features/user/user.api.ts
PROFILE_IMAGE PATCH 호출 리포맷(헤더 유지). 요청/응답 시그니처와 동작 변경 없음.
Pages (JSX) 변경
src/pages/Auth/LoginPage.tsx, src/pages/Auth/OnboardingPage.tsx, src/pages/ComponentGuide/ComponentGuidePage.tsx, src/pages/ComponentGuide/index.ts
JSX 구조·마크업 재배치 및 버튼 태그 수정(자식 텍스트로 변경), 컴포넌트 태그 포맷 정리, 세미콜론 제거 등 포맷 수정. UI/행동 변경 없음.
Store / Hooks / 기타 포맷
src/store/globalModalStore.ts, src/features/meetings/hooks/useMeetingApprovals.ts
함수 시그니처/표현식 줄바꿈·포맷 재정렬(예: openConfirm 라인 래핑, isValidGatheringId 한줄화). 기능 불변.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

feat

Suggested reviewers

  • choiyoungae
🚥 Pre-merge checks | ✅ 2 | ❌ 3
❌ Failed checks (3 warnings)
Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR includes multiple unrelated changes beyond the CI workflow objective, including formatting fixes, JSX syntax corrections, and code restructuring. Remove out-of-scope changes (interceptors.ts, retry.ts, user.api.ts, LoginPage.tsx, OnboardingPage.tsx, ComponentGuidePage.tsx, globalModalStore.ts) and focus only on the CI workflow addition.
Out of Scope Changes check ⚠️ Warning Significant out-of-scope changes detected: CI workflow setup is only ~39 lines, but PR includes 12+ file modifications unrelated to issue #42 objectives. Create a separate PR for formatting/syntax fixes. Keep this PR focused on .github/workflows/ci.yml creation only.
Docstring Coverage ⚠️ Warning Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항을 정확히 설명하고 있습니다. 추가된 CI workflow가 PR 관련이라는 점을 명확히 드러내며, 실제 변경사항의 핵심과 일치합니다.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/add-pr-ci-42

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.github/workflows/ci.yml:
- Around line 1-10: The workflow "CI" triggered by on: pull_request currently
lacks a permissions block, leaving GITHUB_TOKEN with broad defaults; add a
top-level permissions mapping in the CI workflow to minimize scope (e.g., set
contents: read and pull-requests: read and any other specific scopes your jobs
require) so the GITHUB_TOKEN is read-only for this pull_request workflow; update
the .github/workflows/ci.yml file (workflow name "CI") to include the
permissions block rather than relying on defaults.
🧹 Nitpick comments (1)
.github/workflows/ci.yml (1)

16-25: 액션을 커밋 SHA로 고정하세요

문제: actions/*@v4, pnpm/action-setup@v4가 태그로 고정되어 있습니다.
영향: 태그가 이동될 경우 공급망 리스크가 증가합니다.
대안: GitHub 보안 모범 사례에 따라 검증된 커밋 SHA로 고정하세요. Dependabot을 통해 자동 갱신을 설정하면 보안 업데이트를 놓치지 않을 수 있습니다.

- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
- uses: pnpm/action-setup@e0a2c475a6a6de2d9cd10e42a2fcc7b4e5d1cccb # v4.0.0

@mgYang53 mgYang53 changed the title feat: add PR CI workflow feat: PR 관련 CI workflow 추가 Feb 1, 2026
최소 권한 원칙 적용
@mgYang53 mgYang53 merged commit 3dbf290 into main Feb 1, 2026
2 checks passed
@mgYang53 mgYang53 deleted the feat/add-pr-ci-42 branch February 1, 2026 13:04
- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: '20'
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Node.js: >=20.19.0 <21

20.19.0 이상이어야 해여

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.

[feat] PR CI 워크플로우 추가

3 participants