Skip to content

Conversation

sungik-choi
Copy link
Contributor

@sungik-choi sungik-choi commented Dec 17, 2024

Self Checklist

  • I wrote a PR title in English and added an appropriate label to the PR.
  • I wrote the commit message in English and to follow the Conventional Commits specification.
  • I added the changeset about the changes that needed to be released. (or didn't have to)
  • I wrote or updated documentation related to the changes. (or didn't have to)
  • I wrote or updated tests related to the changes. (or didn't have to)
  • I tested the changes in various browsers. (or didn't have to)
    • Windows: Chrome, Edge, (Optional) Firefox
    • macOS: Chrome, Edge, Safari, (Optional) Firefox

Related Issue

Summary

bezier-tokens에서 scss 파일을 내보냅니다.

Details

  • 일부 유틸 클래스를 만드는 CSS 모듈에서 토큰에 대한 종류를 모두 알고있어야하는 불편함이 있었습니다. 이번에 AlphaIcon component #2566 PR 작업을 진행하면서 동일한 작업이 반복되는걸 느껴 이를 함께 수정합니다.
  • bezier-tokens에서 map-deep 포맷으로 scss output을 만들도록 합니다. 자바스크립트 케이스처럼 index.scss 를 만드는 과정을 추가했습니다.
  • bezier-token의 sideEffects 필드에 scss 파일을 추가합니다.
  • scss의 pkg: Importer 규칙에 따라 bezier-tokens의 conditional export field를 수정했습니다. bezier-react에선 storybook & build 과정 모두 상대 경로로 지정하는 게 잘 동작해서 pkg 프로토콜은 이 PR에서 사용하지 않았습니다 (시간이 좀 더 걸릴듯함)

Breaking change? (Yes/No)

No

  • ev-inner, ev-base 에 대한 유틸 클래스가 추가로 생기지만 사용처 영향은 없습니다 (매우 미미한 스타일 시트 파일 크기 상승)

References

@sungik-choi sungik-choi self-assigned this Dec 17, 2024
Copy link

channeltalk bot commented Dec 17, 2024

Copy link

changeset-bot bot commented Dec 17, 2024

🦋 Changeset detected

Latest commit: 3039dcf

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

This PR includes changesets to release 4 packages
Name Type
@channel.io/bezier-tokens Minor
@channel.io/bezier-react Patch
@channel.io/stylelint-bezier 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

@sungik-choi sungik-choi added the bezier-tokens Issue or PR related to bezier-tokens label Dec 17, 2024
Copy link

codecov bot commented Dec 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.88%. Comparing base (bc4319e) to head (3039dcf).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2568   +/-   ##
=======================================
  Coverage   81.88%   81.88%           
=======================================
  Files         145      145           
  Lines        2887     2887           
  Branches      912      912           
=======================================
  Hits         2364     2364           
  Misses        493      493           
  Partials       30       30           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

github-actions bot commented Dec 17, 2024

Chromatic Report

🚀 Congratulations! Your build was successful!

@sungik-choi sungik-choi merged commit d89ac74 into channel-io:main Dec 18, 2024
9 checks passed
@sungik-choi sungik-choi deleted the web-6033 branch December 18, 2024 03:39
sungik-choi pushed a commit that referenced this pull request Dec 19, 2024
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 SCSS support to access design tokens directly through SCSS
variables.
([#2568](#2568)) by
@sungik-choi

## @channel.io/[email protected]

### Patch Changes

- Export the `alphaTokens`, `AlphaTokens`, and `useAlphaTokens` modules.
([#2564](#2564)) by
@sungik-choi

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

## @channel.io/[email protected]

### Patch Changes

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

## [email protected]

### Minor Changes

- Fixed bug where autocomplete did not work.
([#2507](#2507)) by
@yangwooseong

### Patch Changes

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

## [email protected]

### Patch Changes

-   Updated dependencies
    -   @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
bezier-tokens Issue or PR related to bezier-tokens
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants