Merged
Conversation
포크한 레포에 작동 테스틀 위해 연결 스토리북 형태도 세팅함
구조 수정 및 tailwind prefix에 맞게 mapping
이전 패키지 버전에선 빌드가 안 됐음.
code형태 아니어야 했기에.
숫자 표기 안되는 문제로 인해.
다른 프로젝트랑 연결 되어 있었기에 수정.
forked에서의 pull request로는 secret을 사용할 수 없기 때문에.
파싱되지 않던 형식 처리, tailwind에선 사용되지 않는 토큰 제거, tailwind prefix 매핑 수정.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
크로마틱 CI 에러 관련 이슈
이 PR을 올릴 때, 크로마틱 CI 에러 관련 이슈가 있었습니다.
chromaticui/action의 버전을 잘못 기입해 발생했습니다.
이를 고치기 위해 해당 CI 에러를 무시하고 merge하고자 합니다.
이유는 다음과 같습니다.
이 PR에서는 컴포넌트 UI 변경이 없기에 문제되지 않습니다.
chromatic관련 yml은 pull_request_target에 트리거 되기 때문에, upstream/main의 yml파일을 사용해 github action이 동작합니다.
즉, 이전 PR에서 잘못 올린 chromatic관련 yml으로 인해 비정상 ci동작이 일어난 것입니다.
이번 PR이후엔 정상 작동하게 됩니다.
상황
기존 토큰 형태와 다른 형태가 존재하여 파싱 동작을 수정했습니다.
tailwind를 사용하기에 필요 없는 semantic token들이 존재했고,
그 중 parsing에 에러를 발생시키는 토큰을 제외했습니다.
(title, body, caption)
변경한 파서로 토큰을 재생성 했습니다.