-
Notifications
You must be signed in to change notification settings - Fork 1
feat: 디자인 토큰 연결 #11
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: 디자인 토큰 연결 #11
Conversation
Summary of ChangesHello @dasosann, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 Pull Request는 프로젝트에 디자인 토큰 시스템을 통합하기 위한 첫 번째 단계입니다. 피그마 디자인 토큰을 Git과 연결하기 위한 기반을 마련하며, Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
이 PR은 Figma 디자인 토큰을 연동하기 위한 초기 설정으로 tokens/token.json 파일을 추가합니다. 파일 구조는 디자인 토큰 관리를 위한 표준적인 형태로 보입니다. 다만, 파일의 마지막에 개행 문자를 추가하여 POSIX 표준을 준수하고 도구 호환성을 높이는 것을 제안합니다. 전반적으로 디자인 시스템 통합을 위한 좋은 첫걸음입니다.
tokens/token.json
Outdated
| { | ||
| "global": {}, | ||
| "$themes": [], | ||
| "$metadata": { | ||
| "tokenSetOrder": [ | ||
| "global" | ||
| ] | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
파일의 마지막에 개행 문자를 추가하는 것을 권장합니다. 이는 POSIX 표준을 따르는 좋은 관행이며, 일부 도구에서 예기치 않은 동작을 방지할 수 있습니다.1
| { | |
| "global": {}, | |
| "$themes": [], | |
| "$metadata": { | |
| "tokenSetOrder": [ | |
| "global" | |
| ] | |
| } | |
| } | |
| { | |
| "global": {}, | |
| "$themes": [], | |
| "$metadata": { | |
| "tokenSetOrder": [ | |
| "global" | |
| ] | |
| } | |
| } | |
Rules References
Footnotes
-
파일은 개행 문자로 끝나야 합니다. 이는 POSIX 호환성을 보장하고, 파일 결합이나
cat과 같은 도구 사용 시 발생할 수 있는 문제를 방지합니다. ↩
이슈 넘버
#10
구현 사항
Need Review
Figma에서 Token Studio 플러그인을 이용해서 color-token 브랜치로 token.json을 생성해주면
그 token,json을 받아서 pnpm run token 명령어를 통해 generate-color-tokens.mjs를 통해 global.css로 변환합니다
이제 나중에 색상 사용하실때는 text-17-500, 또는 bg-button-primary-fill 이런식으로 사용해주시면 될 것 같습니다.
추가로 만약에 토큰을 직접 변경하시려고 한다면 기존에 있던 token.json 내용은 지우고 token studio에서 재 발급 받아야합니다!
📸 스크린샷
Reference
📜 리뷰 규칙
Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.