-
Notifications
You must be signed in to change notification settings - Fork 1
Chore/#1 storybook 및 chromatic 배포 설정 #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR sets up Storybook for component development and configures Chromatic deployment for visual testing, while also simplifying the pre-commit process by replacing lint-staged with direct pnpm lint execution.
- Sets up Storybook with default components (Button, Header, Page) and configuration
- Configures Chromatic deployment workflow for automated visual testing on pull requests
- Simplifies pre-commit hooks by removing lint-staged dependency and using pnpm lint directly
Reviewed Changes
Copilot reviewed 21 out of 44 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Removes lint-staged configuration and dependency |
| lint-staged.config.js | Completely removes lint-staged configuration file |
| lefthook.yml | Updates pre-commit to use pnpm lint instead of lint-staged |
| apps/storybook/* | Adds complete Storybook setup with stories, configuration, and build scripts |
| .github/workflows/deploy-chromatic.yml | Configures automated Chromatic deployment for visual testing |
Comments suppressed due to low confidence (1)
- GitHub Actions에 pnpm과 Node.js를 한 번에 설치하는 composite action을 도입 - Chromatic 워크플로에서 중복된 setup-node 단계를 제거하고 새 액션을 사용하도록 변경
#️⃣연관된 이슈
📝작업 내용
스크린샷 (선택)
💬리뷰 요구사항(선택)