search

channel-io / bezier-react

React components library that implements Bezier design system.
https://main--62bead1508281287d3c94d25.chromatic.com
Apache License 2.0
184 stars 46 forks source link

Deploy bezier-react storybook when the package is published #838

Closed inhibitor1217 closed 1 year ago

inhibitor1217 commented 1 year ago

Summary

@channel.io/bezier-react package publish 시, storybook을 빌드하여 GitHub Pages로 배포합니다.

Details

Points

~Browser Compatibility~

References

codecov[bot] commented 1 year ago

Codecov Report

Merging #838 (b5e86c9) into next-v1 (3478234) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff            @@
##           next-v1     #838   +/-   ##
========================================
  Coverage    67.97%   67.97%           
========================================
  Files          204      204           
  Lines         2860     2860           
  Branches       784      784           
========================================
  Hits          1944     1944           
  Misses         800      800           
  Partials       116      116

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3478234...b5e86c9. Read the comment docs.

sungik-choi commented 1 year ago

Storybook에서 release note를 쌓아 보여주는 feature가 있었으면 좋겠음.

동의합니다.

sungik-choi commented 1 year ago

논의를 거쳐서 후에 아이콘 패키지가 추가된다거나하여 진짜 멀티 패키지의 형태를 갖추게 되면, https://github.com/changesets/changesets 를 적용하면서 Changesets이 만들어주는 CHANGELOG.md 를 스토리북에 띄워주는 방식이 되면 잘 동작할 거 같다는 생각이 들었어요

sungik-choi commented 1 year ago

845 와 중복 구현되는 부분은 아닐지 확인 필요.

sungik-choi commented 1 year ago

논의를 거쳐서 후에 아이콘 패키지가 추가된다거나하여 진짜 멀티 패키지의 형태를 갖추게 되면, https://github.com/changesets/changesets 를 적용하면서 Changesets이 만들어주는 CHANGELOG.md 를 스토리북에 띄워주는 방식이 되면 잘 동작할 거 같다는 생각이 들었어요

https://github.com/channel-io/bezier-react/blob/next-v1/packages/bezier-react/CHANGELOG.md 더 빨리 도입하게 되어서, 생각보다 빠르게 적용해볼 수 있을지도 모르겠네요

inhibitor1217 commented 1 year ago

@sungik-choi

제가 이해하기로는, chromatic은 브랜치별로 스토리북을 빌드해서 chromatic에서 제공하는 도메인에서 serve해주는 역할까지 담당하는 것입니다 (#845)

아마 추가로 작업이 이루어질 부분은, 스토리북을 빌드해서 github pages로 배포하는 부분일 텐데, #845 와는 독립적으로 가도 상관없다고 생각하긴 합니다. 혹시 생각해두신 부분 있으시면 말씀 주세요!

sungik-choi commented 1 year ago

@inhibitor1217

https://next-v1--62bead1508281287d3c94d25.chromatic.com/?path=/story/intro--page 이 주소로 항상 next-v1 최신 커밋으로 배포된 상태이긴해서, 깃헙 페이지를 이 페이지로 리다이렉트시키면 어떨까 생각하고 있었어요. release 워크플로우에서 deploy:storybook 실행시켜도 괜찮을 거 같기도 하구요. 어떻게 생각하시나요?

inhibitor1217 commented 1 year ago

아하 좋습니다! 그러면 이 pr은 닫아도 좋을것 같습니다!