Skip to content
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

feat: deploy storybook #14

Merged
merged 3 commits into from
Nov 12, 2023
Merged

feat: deploy storybook #14

merged 3 commits into from
Nov 12, 2023

Conversation

Seojunhwan
Copy link
Collaborator

PR의 목적을 알려주세요.

스토리북을 배포(테스트)합니다!

어떤 변경사항이 있는지 알려주세요.

  • 스토리북 배포용 CI 가 추가되었습니다. (메인에 병합되었을 때 트리거 됩니다)
  • ads-react의 개발 의존성에 스토리북이 추가되었습니다.

중점적으로 봐주었으면 하는 부분

CI가 잘 이행될까요?

@Seojunhwan Seojunhwan self-assigned this Nov 12, 2023
Comment on lines +28 to +29
yarn install --immutable
yarn lerna run build --stream --ignore next
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

next는 필요 없어서 제외합니다

return dirname(require.resolve(join(value, "package.json")));
}
const config: StorybookConfig = {
stories: ["../../**/*.stories.@(js|jsx|mjs|ts|tsx)"],
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

요게 ads에서 타 패키지들을 종합하여 배포하는 부분입니다

Copy link
Owner

@sjoleee sjoleee left a comment

Choose a reason for hiding this comment

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

굿굿 좋으네여 지금은 TailwindTestComponent의 스타일이 스토리북에서 적용이 안되는데, 요거슨 ads-react에 테일윈드 의존성 추가하고 tailwind.config에 아래처럼 코드 추가해주면 되네요 ㅋㅋㅋㅋㅋ 거의 만병통치약;

const packagePath = require.resolve("@sjoleee/tailwind-test-component");
const packageDir = dirname(packagePath);

위 경로 찾아주는 코드는 진짜 컴포넌트 개발 시작하면 추가하는걸로 하고 머지 고고하시죠

@Seojunhwan Seojunhwan merged commit 18bd1c7 into main Nov 12, 2023
3 checks passed
@Seojunhwan Seojunhwan deleted the feat/storybook-deploy branch November 12, 2023 09:21
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.

3 participants