-
Notifications
You must be signed in to change notification settings - Fork 1
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
⚙️ storybook 파이프라인 구축 #4
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BangDori
reviewed
Oct 12, 2024
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.
디스코드에 업로드된 chromatic invite link 확인했습니다! storybook-deployment 단계에서 확인해보고 싶은 게 있어서 Comment 하였습니다!
suhwan2004
force-pushed
the
feat/storybook-pipeline
branch
2 times, most recently
from
October 12, 2024 11:00
dd772fc
to
2a6e3c4
Compare
suhwan2004
force-pushed
the
feat/storybook-pipeline
branch
from
October 12, 2024 11:07
2a6e3c4
to
25ade29
Compare
suhwan2004
force-pushed
the
feat/storybook-pipeline
branch
from
October 12, 2024 11:31
5a69cd1
to
e92cce2
Compare
suhwan2004
force-pushed
the
feat/storybook-pipeline
branch
from
October 12, 2024 11:42
7cbe281
to
5ec1562
Compare
BangDori
previously approved these changes
Oct 12, 2024
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.
고생하셨습니다~~!!
suhwan2004
requested review from
BangDori and
Legitgoons
and removed request for
Legitgoons
October 12, 2024 12:28
BangDori
previously approved these changes
Oct 12, 2024
suhwan2004
force-pushed
the
feat/storybook-pipeline
branch
2 times, most recently
from
October 12, 2024 14:18
5170662
to
f0f001d
Compare
suhwan2004
force-pushed
the
feat/storybook-pipeline
branch
from
October 12, 2024 14:21
f0f001d
to
45b8c9f
Compare
🔍 Visual review for your branch is published 🔍Here are the links to: |
suhwan2004
requested review from
Legitgoons
and removed request for
Legitgoons
October 12, 2024 14:24
Legitgoons
approved these changes
Oct 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 이유
작업 사항
1️⃣ yml 파일 생성
storybook-test-and-deploy.yml이라는 네이밍으로 파일을 생성하였습니다.
해당 플로우의 경우, install-and-test.yml을 어느정도 참조하여 상당부분이 비슷하나, 차이점은 다음과 같습니다.
가장 큰 차이점은 lint까지 똑같이 하나, 마지막 단계에서 단위테스트 대신 chromatic을 사용하여 storybook을 배포하고 있습니다.
chromatic을 통한 배포 시에, commit이 하나가 있는 경우 에러가 발생합니다.
해당 사항 방지를 위해 부모 브랜치인 main의 commit 까지 참조하기 위해서, step > Checkout에서 with: fetch-depth: 0 을 추가했습니다.
on 확인 시에 push로 해두었는데 push 내 component.tsx 및 component.stories.tsx 가 존재할 시 이를 배포 페이지에서 storybook으로보여줘야 되지 않나 생각하여 추가해두었습니다.
2️⃣ storybook 세팅
docs에서 제공되는 커멘드를 통한 기본 설치 및 프로젝트에 필요해보이는 addon(vscode의 익스텐션 같은 느낌입니다)들을 다운로드 하였습니다.
기본 설치된 addon 제외 부가 설치한 에드온들의 리스트 및 설치 이유에 대해 말씀드리고자 합니다.
=> .storybook/preview.ts를 확인 시 해당 addon 관련 추가세팅이 있습니다. 현재는 아이폰 14 프로 멕스입니다.
리뷰어가 중점적으로 확인해야 하는 부분
발견한 이슈
@storybook/addon-storysource 이슈
=> 위에서 말한 이 라이브러리의 경우, 현재 컴포넌트 생성 후에 확인해보심 실제 tsx 컴포넌트가 아니라, storeis.tsx의 코드가 나와버립니다. 의도랑 상당히 다르게 동작하는데 현재 저희 스토리북이 8.3.5인데 6.5까지 내리면 된다는 내용이 있었습니다. storybookjs/storybook#22922 여기를 보시면 자세히 확인 가능하며 아직까지 현재진행형으로 보입니다...
chromatic 초대 관련
=> 아마 제가 생성한 프로젝트 페이지에 초대를 받아야 배포된 스토리북이 확인 될 것 같습니다. 해당 링크의 경우 discord에 공지하겠습니다.