Skip to content

Conversation

@wes5510
Copy link
Contributor

@wes5510 wes5510 commented Nov 28, 2025

Icon update is ready to be merged! 🎉

  • 2 icon(s) modified
  • 7 icon(s) added
  • 1 icon(s) deleted
Name Status
area-chart.svg ✏️
line-chart.svg ✏️
combo-chart.svg 🆕
page-add.svg 🆕
pathfinder-exclude.svg 🆕
pathfinder-intersect.svg 🆕
pathfinder-subtract-left.svg 🆕
pathfinder-subtract-right.svg 🆕
pathfinder-union.svg 🆕
intersect.svg 🗑️

Summary by CodeRabbit

  • 새로운 기능
    • 조합 차트, 페이지 추가, 경로 파인더 관련 아이콘 등 새로운 아이콘 7개가 추가되었습니다.
    • 기존 차트 관련 아이콘이 업데이트되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@changeset-bot
Copy link

changeset-bot bot commented Nov 28, 2025

🦋 Changeset detected

Latest commit: 3a79512

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-icons Minor
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

@channeltalk
Copy link

channeltalk bot commented Nov 28, 2025

@coderabbitai
Copy link

coderabbitai bot commented Nov 28, 2025

개요

Bezier 아이콘 패키지의 마이너 버전을 업데이트하여 차트 및 경로 편집 관련 아이콘을 추가하고, 기존 차트 아이콘을 수정하며, 중복된 intersect.svg를 삭제합니다.

변경 사항

분류 / 파일 변경 내용
차트 아이콘
area-chart.svg, line-chart.svg
기존 차트 아이콘 수정
새로운 차트 아이콘
combo-chart.svg
새로운 복합 차트 아이콘 추가
경로 편집 도구 아이콘
pathfinder-exclude.svg, pathfinder-intersect.svg, pathfinder-subtract-left.svg, pathfinder-subtract-right.svg, pathfinder-union.svg
경로 편집 작업용 새로운 아이콘 세트 추가
기타 아이콘
page-add.svg
새로운 페이지 추가 아이콘
삭제된 아이콘
intersect.svg
중복 아이콘 제거
Changeset
.changeset/icon-update-1764303366516.md
패키지 버전 업데이트 선언

예상 검토 노력

🎯 1 (Trivial) | ⏱️ ~2분

  • 아이콘 자산 업데이트로 로직 변경이 없으며, 구조적 영향이 제한적입니다.

관련 PR 목록

축하 시

🐰✨ 새로운 아이콘들이 반짝반짝
차트와 경로들이 춤을 춘단다
중복된 친구는 안녕~👋
Bezier의 창고가 더욱 예뻐졌어요!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명이 변경사항 목록을 제공하지만, 필수 템플릿 섹션(Self Checklist, Related Issue, Summary, Details, Breaking change, References)이 대부분 누락되어 있습니다. 변경 이유와 상세 설명이 부족합니다. 제목줄 규칙, Self Checklist, Related Issue, Summary, Details, Breaking change 섹션을 포함하여 템플릿을 완성하세요. 특히 변경의 목적과 이유를 명확히 설명해주세요.
Title check ❓ Inconclusive PR 제목이 변경 사항의 주요 내용을 명확하게 설명하지 못하고 있습니다. 'Icon update PR made by bezier-asset main'는 아이콘 업데이트의 구체적인 내용(어떤 아이콘이 추가/수정/삭제되었는지)을 명시하지 않아 모호합니다. PR 제목을 더 구체적으로 수정하세요. 예: 'chore: Update icons with new chart and pathfinder components' 또는 'feat: Add new icons and update existing ones'과 같이 Conventional Commits 형식을 따르면서 변경 내용을 명확히 해주세요.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch icon-update-main

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Nov 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.33%. Comparing base (308c318) to head (3a79512).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2728   +/-   ##
=======================================
  Coverage   79.33%   79.33%           
=======================================
  Files         146      146           
  Lines        2913     2913           
  Branches      932      932           
=======================================
  Hits         2311     2311           
+ Misses        598      567   -31     
- Partials        4       35   +31     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
.changeset/icon-update-1764303366516.md (1)

5-5: 변경로그 설명을 더 자세하게 작성하는 것을 권장합니다.

현재 "Update icons"는 매우 간결합니다. 추가되는 차트 관련 아이콘들(combo-chart)이나 패스파인더 기능(pathfinder) 등에 대한 더 자세한 설명을 포함하면 사용자가 변경사항을 더 쉽게 이해할 수 있습니다. 예: "차트 및 경로 도구 아이콘 추가"

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 308c318 and 3a79512.

⛔ Files ignored due to path filters (10)
  • packages/bezier-icons/icons/area-chart.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/combo-chart.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/intersect.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/line-chart.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/page-add.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/pathfinder-exclude.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/pathfinder-intersect.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/pathfinder-subtract-left.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/pathfinder-subtract-right.svg is excluded by !**/*.svg
  • packages/bezier-icons/icons/pathfinder-union.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • .changeset/icon-update-1764303366516.md (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Chromatic
🔇 Additional comments (1)
.changeset/icon-update-1764303366516.md (1)

1-24: 변경집합 파일이 올바르게 구성되었습니다.

이 변경집합 파일은 형식이 유효하고 모든 아이콘 변경사항이 정확하게 문서화되어 있습니다. 마이너 버전 범프는 새로운 아이콘 추가 및 기존 아이콘 수정에 적절합니다.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

Chromatic Report

🚀 Congratulations! Your build was successful!

@annie1229 annie1229 merged commit 6635742 into main Nov 28, 2025
7 checks passed
@annie1229 annie1229 deleted the icon-update-main branch November 28, 2025 07:27
annie1229 pushed a commit that referenced this pull request Nov 28, 2025
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

- Update icons
([#2728](#2728)) by
@wes5510

    Modified

    -   area-chart.svg
    -   line-chart.svg

    Added

    -   combo-chart.svg
    -   page-add.svg
    -   pathfinder-exclude.svg
    -   pathfinder-intersect.svg
    -   pathfinder-subtract-left.svg
    -   pathfinder-subtract-right.svg
    -   pathfinder-union.svg

    Deleted

    -   intersect.svg

## [email protected]

### Patch Changes

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


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

## 릴리스 노트

* **새로운 기능**
  * 콤보 차트, 페이지 추가, 경로찾기 관련 아이콘 7개 추가
  * 차트 아이콘 업데이트

* **변경 사항**
  * 기존 아이콘 제거 및 개선
  * 아이콘 라이브러리 버전 0.52.0으로 업데이트

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: github-actions[bot] <41898282+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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants