Skip to content

Conversation

@Dogdriip
Copy link
Contributor

@Dogdriip Dogdriip commented Sep 5, 2023

Self Checklist

  • I wrote a PR title in English.
  • I added an appropriate label to the PR.
  • I wrote a commit message in English.
  • I wrote a commit message according to the Conventional Commits specification.
  • I added the appropriate changeset for the changes.
  • [Component] I wrote a unit test about the implementation.
  • [Component] I wrote a storybook document about the implementation.
  • [Component] I tested the implementation in various browsers.
    • Windows: Chrome, Edge, (Optional) Firefox
    • macOS: Chrome, Edge, Safari, (Optional) Firefox
  • [New Component] I added my username to the correct directory in the CODEOWNERS file.

Related Issue

Summary

  • Apply react-docgen-typescript-plugin to generate storybook documentations from JSDoc #931 에서 react-docgen-typescript-plugin을 적용하였으나, 이 플러그인이 react-docgen-typescript v1을 사용하는 이슈로 인해 이를 fork한 별도의 플러그인 @channel.io/react-docgen-typescript-plugin을 사용하고 있었습니다.
  • react-docgen-typescript-pluginreact-docgen-typescript v2를 사용하게 됨에 따라, 별도의 플러그인을 제거하고 내장 플러그인을 사용하도록 합니다.

Details

  • 이외의 의도한 동작(OOM 이슈로 production에서만 동작하도록 함) 등은 유지합니다.

Breaking change or not (Yes/No)

  • No

References

@Dogdriip Dogdriip added the documentation Issue or PR that improvements or additions to documentation label Sep 5, 2023
@Dogdriip Dogdriip self-assigned this Sep 5, 2023
@changeset-bot
Copy link

changeset-bot bot commented Sep 5, 2023

🦋 Changeset detected

Latest commit: 11a1305

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@channel.io/bezier-react Patch
bezier-figma-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (848167b) 87.32% compared to head (11a1305) 87.32%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1594   +/-   ##
=======================================
  Coverage   87.32%   87.32%           
=======================================
  Files         280      280           
  Lines        3843     3843           
  Branches      806      806           
=======================================
  Hits         3356     3356           
  Misses        414      414           
  Partials       73       73           

☔ View full report in Codecov by Sentry.

📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2023

Chromatic Report

🚀 Congratulations! Your build was successful!

@Dogdriip Dogdriip changed the title Replace @channel.io/react-docgen-typescript-plugin with Storybook's reactDocgen option Replace react-docgen-typescript-plugin with Storybook's reactDocgen option Sep 8, 2023
@Dogdriip Dogdriip merged commit 2d6b5f9 into channel-io:main Sep 8, 2023
@Dogdriip Dogdriip deleted the fix/use-storybook-docgen-plugin branch September 8, 2023 02:47
sungik-choi pushed a commit that referenced this pull request Sep 12, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @channel.io/[email protected]

### Minor Changes

- Add Banner to iconName transformation target
([#1562](#1562)) by
@yangwooseong

## @channel.io/[email protected]

### Minor Changes

- Add script for generating svg file from icons.json
([#1596](#1596)) by
@yangwooseong

## @channel.io/[email protected]

### Minor Changes

- Allow `iconName` prop of `Button`, `Banner`, `SectionLabel` component
to include `BezierIcon` type
([#1562](#1562)) by
@yangwooseong

### Patch Changes

- Fixed a rendering bug that occurs when using `react-resize-detector`
and `asChild`prop of `radix-ui` simultaneously.
([#1577](#1577)) by
@yangwooseong

- Removed indicator adjusting logic by `react-resize-detector` using css
transform property

- Replace `@channel.io/react-docgen-typescript-plugin` with Storybook's
`reactDocgen` option
([#1594](#1594)) by
@Dogdriip

## [email protected]

### Minor Changes

- Enhance bezier-figma-plugin running performance
([#1596](#1596)) by
@yangwooseong
- Remove svg extract process using FigmaAPI and merely send json file
that contains svg string
    -   Make icon files based on given json file during Github Action

### Patch Changes

-   Updated dependencies
    -   @channel.io/[email protected]
    -   @channel.io/[email protected]

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Issue or PR that improvements or additions to documentation

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants