Skip to content
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

Icon update PR made by bezier-asset main #2356

Closed
wants to merge 1 commit into from
Closed

Conversation

ch-builder
Copy link
Contributor

Icon update PR made by bezier-asset

Copy link

changeset-bot bot commented Jul 15, 2024

⚠️ No Changeset found

Latest commit: 13f6ecb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

Copy link

channeltalk bot commented Jul 15, 2024

@yangwooseong
Copy link
Collaborator

bezier-asset 에서 최신 아이콘으로 업데이트하고 다시 트리거 하겠습니다

yangwooseong added a commit that referenced this pull request Jul 17, 2024
<!--
  How to write a good PR title:
- Follow [the Conventional Commits
specification](https://www.conventionalcommits.org/en/v1.0.0/).
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

## Self Checklist

- [x] I wrote a PR title in **English** and added an appropriate
**label** to the PR.
- [x] I wrote the commit message in **English** and to follow [**the
Conventional Commits
specification**](https://www.conventionalcommits.org/en/v1.0.0/).
- [x] I [added the
**changeset**](https://github.com/changesets/changesets/blob/main/docs/adding-a-changeset.md)
about the changes that needed to be released. (or didn't have to)
- [x] I wrote or updated **documentation** related to the changes. (or
didn't have to)
- [x] I wrote or updated **tests** related to the changes. (or didn't
have to)
- [x] 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

<!-- Please link to issue if one exists -->

<!-- Fixes #0000 -->

- #2229 

## Summary

<!-- Please brief explanation of the changes made -->

- 피그마 플러그인에 의해 만들어진 icons.json에 따라 아이콘 svg파일을 만드는 스텝을 액션에서 제거합니다. 

## Details

<!-- Please elaborate description of the changes -->

- #2357 에서 피그마 플러그인이 바라보는
레포가 bezier-react에서 bezier-asset으로 변경되면 bezier-react에서는 icons.json파일이
만들어지지 않고 아이콘 파일이 직접 생성되게 됩니다. (e.g.
#2356)
- 이에 따라 기존의 generate-icon-files.yml 에서 icons.json파일 관련 스텝을 없애고 액션 이름을
변경합니다.

### Breaking change? (Yes/No)

<!-- If Yes, please describe the impact and migration path for users -->

- No

## References

<!-- Please list any other resources or points the reviewer should be
aware of -->
@yangwooseong yangwooseong deleted the icon-update-main branch July 23, 2024 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants