Conversation
- main, develop-fe 브랜치 대상 PR 시 실행 - Prettier check, ESLint, Build 순서로 검증 - frontend/ 경로 변경 시에만 트리거
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| Cohort / File(s) | Summary |
|---|---|
CI 워크플로우 추가 /.github/workflows/frontend-ci.yml |
프론트엔드 디렉토리 변경 시 PR에서 트리거되는 Frontend CI 파이프라인 추가: checkout → Node.js(v20) 셋업( npm 캐시 ) → npm ci → Prettier 체크 → lint → build. |
Prettier 설정 frontend/.prettierignore, frontend/.prettierrc |
public/mockServiceWorker.js를 Prettier 무시 대상에 추가하고, importOrderSortSpecifiers 옵션 제거(수정된 포맷팅 규칙). |
UI 컴포넌트 및 페이지 포맷팅/리팩토링 frontend/src/pages/.../ClubDetailTopBar.tsx, .../ClubDetailTopBar.styles.ts, .../LegacyClubDetailPage.tsx, .../ClubUnionPage.tsx, .../MainPage/.../Banner.tsx, .../CategoryButtonList.stories.tsx, .../ClubCard.stories.tsx |
임포트 재정렬, 상수 추가(SCROLL_THRESHOLD 및 useScrollTrigger 연동), JSX/스타일 객체 재포맷팅 및 소규모 스타일/문자열 포맷 변경(기능적 변경 없음). |
훅 / 유틸리티 포맷 변경 frontend/src/hooks/Scroll/useScrollTo.ts, frontend/src/utils/linkifyText.tsx, frontend/src/utils/linkifyText.test.tsx, frontend/src/utils/webviewBridge.ts |
함수 시그니처/파라미터 줄바꿈 및 JSX/테스트 포맷 정리, 불필요 공백 제거(동작 미변경). |
페이지 탭/폼 업데이트(포맷) frontend/src/pages/AdminPage/.../ApplicationEditTab.tsx |
update mutation의 onSuccess 내부 invalidateQueries 호출 인자 포맷 재배치(동작 미변경). |
기타 경미한 변경 frontend/src/pages/ClubDetailPage/components/ClubFeed/ClubFeed.tsx, frontend/src/pages/ClubDetailPage/components/ClubIntroContent/LinkifiedText.tsx |
임포트 순서/따옴표 스타일 및 파일 끝 개행 추가 등 경미한 포맷 변경. |
Sequence Diagram(s)
sequenceDiagram
participant PR as Pull Request
participant GH as GitHub Actions
participant Runner as ubuntu-latest Runner
participant Node as Node/npm
participant Prettier as Prettier
participant ESLint as ESLint
participant Build as Build Step
PR->>GH: PR to main/develop-fe affecting frontend
GH->>Runner: start job (working-directory: frontend)
Runner->>Node: setup Node.js v20, npm cache (actions/setup-node)
Runner->>Node: run `npm ci`
Runner->>Prettier: run Prettier check (specified file globs)
Prettier-->>Runner: pass/fail
Runner->>ESLint: run lint
ESLint-->>Runner: pass/fail
Runner->>Build: run build
Build-->>GH: report status (success/failure)
GH-->>PR: update PR status
(각 박스 색상 표현이 필요한 경우 도구에서 렌더링 시 rgba(100,149,237,0.5) 등으로 스타일을 적용하세요.)
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
- [release] FE v1.1.21 #1124 — 동일 파일(
ApplicationEditTab.tsx)의 invalidateQueries 호출 포맷 변경과 관련된 편집이 있음. - [release] FE v1.1.19 #1106 —
useScrollTo,ClubDetailTopBar,webviewBridge등 동일 유틸/컴포넌트들을 수정한 기록이 있어 코드 레벨로 연관됨. - [feature] FAQ 답변 텍스트 내 URL을 자동으로 링크 처리한다 #1117 —
linkifyText및 관련 테스트/컴포넌트 변경을 포함, 동일 유틸리티 변경과 연관됨.
Suggested reviewers
- lepitaaar
- seongwon030
- suhyun113
🚥 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 제목이 변경 사항의 주요 내용을 정확하게 반영하고 있습니다. 프론트엔드 CI 워크플로우 추가라는 핵심 변경사항을 명확하게 설명합니다. |
| Linked Issues check | ✅ Passed | PR이 MOA-615의 모든 필수 요구사항을 충족합니다: GitHub Actions 워크플로우 추가, ESLint 검사 단계 구성, Prettier format check 단계 구성, 관련 포맷 정리(deprecated 옵션 제거, 전역 포맷 적용). |
| Out of Scope Changes check | ✅ Passed | 모든 파일 변경사항이 frontend CI 워크플로우 추가 및 관련 코드 포맷팅에 직접적으로 관련되어 있습니다. 범위 밖의 변경사항은 없습니다. |
| 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 docstrings
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
feature/#1155-fe-ci-lint-format-MOA-615
No actionable comments were generated in the recent review. 🎉
🧹 Recent nitpick comments
.github/workflows/frontend-ci.yml (2)
10-15: Job에timeout-minutes설정을 추가하는 것을 권장합니다.빌드가 hang될 경우 기본 타임아웃(6시간)까지 CI 리소스를 소비할 수 있습니다. 일반적인 프론트엔드 CI라면 10~15분 정도면 충분합니다.
♻️ 제안
jobs: ci: runs-on: ubuntu-latest + timeout-minutes: 15 defaults: run: working-directory: frontend
3-8: 동일 PR에 대한 중복 실행 방지를 위해concurrency설정을 고려해 보세요.PR에 새로운 커밋이 push될 때마다 이전 실행이 취소되도록 설정하면 CI 리소스를 절약할 수 있습니다.
♻️ 제안
on: pull_request: branches: [main, develop-fe] paths: - 'frontend/**' - '.github/workflows/frontend-ci.yml' + +concurrency: + group: frontend-ci-${{ github.event.pull_request.number }} + cancel-in-progress: true
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
- MSW가 생성·관리하는 파일이므로 직접 포맷을 수정하면 안 됨
#️⃣연관된 이슈
📝작업 내용
main,develop-fe브랜치 대상 PR 시 프론트엔드 CI가 실행되도록 GitHub Actions 워크플로우를 추가했습니다.Prettier check → ESLint → Build 순서로 검증합니다.
frontend/경로 변경 시에만 트리거됩니다.CI Prettier check 실패에 따라 전역 포맷을 적용했습니다.
@ianvs/prettier-plugin-sort-importsv4.x에서 deprecated된importOrderSortSpecifiers옵션을 제거했습니다.🫡 참고사항
npm run build에tsc --noEmit이 포함되어 있어 타입 체크도 함께 수행됩니다.Summary by CodeRabbit
릴리스 노트