Skip to content

[REFACTOR] 아이콘 추가 시 patch 버전 업데이트되도록 수정#124

Merged
ujinsim merged 3 commits intomainfrom
feat/#123
Dec 1, 2025
Merged

[REFACTOR] 아이콘 추가 시 patch 버전 업데이트되도록 수정#124
ujinsim merged 3 commits intomainfrom
feat/#123

Conversation

@ujinsim
Copy link
Copy Markdown
Collaborator

@ujinsim ujinsim commented Dec 1, 2025

🔥 연관 이슈

🚀 작업 내용

기존 워크플로가 shared/ui/ 하위에 추가된 모든 파일을 새 UI 컴포넌트로 간주했습니다.
아이콘 파일 또한 추가되면 assets가 새 컴포넌트로 인식되어 major 버전이 올라가게 됐습니다

그래서 assets 폴더를 제외하도록 조건 추가해서 major 버전업 안되도록 수정했습니다

🤔 고민했던 내용

💬 리뷰 중점사항

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 버전을 1.0.26으로 업데이트했습니다.
    • 배포 워크플로우의 폴더 감지 로직을 개선하여 특정 디렉토리가 제외되도록 조정했습니다.

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

@ujinsim ujinsim self-assigned this Dec 1, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 1, 2025

주요 내용

아이콘 추가 시 patch 버전만 업데이트되도록 버전 범프 워크플로우의 UI 폴더 감지 로직을 수정하고, 패키지 버전을 업데이트했습니다.

변경 사항

코호트 / 파일 변경 요약
워크플로우 설정
\.github/workflows/bump-and-publish\.yaml
새 UI 폴더 감지 조건에 assets 디렉토리 제외 필터 추가 (parts[3] !== 'assets' 조건 추가)
버전 관리
package\.json
패키지 버전 2.0.0에서 1.0.26으로 업데이트

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • 워크플로우 조건식 수정은 단순 필터링 로직 추가
  • 버전 번호 변경은 매우 간단한 수정

연관된 PR

제안 라벨

chore

제안 검토자

  • keemsebin
  • yougyung

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 '[REFACTOR] 아이콘 추가 시 patch 버전 업데이트 되도록 수정'은 변경사항의 핵심을 명확하게 반영합니다.
Description check ✅ Passed PR 설명이 필수 섹션(연관 이슈, 작업 내용)을 포함하고 있으나, '고민했던 내용'과 '리뷰 중점사항' 섹션이 비어있습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 linked issue #123의 요구사항을 완전히 충족합니다. 워크플로 조건 수정으로 아이콘 추가 시 patch 버전만 업데이트되도록 구현했습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 issue #123의 요구사항 범위 내입니다. 워크플로 수정과 버전 업데이트는 모두 아이콘 추가 시 patch 버전 업데이트 목표와 직접적으로 연관되어 있습니다.
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 feat/#123

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a7163d9 and 877ede2.

📒 Files selected for processing (2)
  • .github/workflows/bump-and-publish.yaml (1 hunks)
  • package.json (1 hunks)
🔇 Additional comments (2)
package.json (1)

3-3: 버전 다운그레이드의 의도 확인 필요

버전이 2.0.0에서 1.0.26으로 다운그레이드되었습니다. 이는 일반적인 semantic versioning 정책과 맞지 않습니다. 다음을 확인해 주세요:

  1. 2.0.0이 이미 npm에 배포되었는지 여부
  2. 버전 다운그레이드가 의도적인 롤백인지, 아니면 다른 버전 관리 전략의 일부인지
  3. 기존 사용자가 2.0.0을 이미 설치했다면 호환성 문제가 발생할 수 있는지
.github/workflows/bump-and-publish.yaml (1)

63-63: assets 폴더 제외 로직이 정확하게 구현됨

parts[3] !== 'assets' 조건이 추가되어 src/shared/ui/assets/ 경로의 파일들이 새 UI 컴포넌트로 감지되지 않도록 올바르게 처리되었습니다. 이를 통해 아이콘 추가 시 major 버전 대신 patch 버전이 업데이트될 수 있게 되었습니다.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@ujinsim ujinsim merged commit 8a32b00 into main Dec 1, 2025
6 checks passed
@ujinsim ujinsim deleted the feat/#123 branch December 1, 2025 08:38
@ujinsim ujinsim changed the title [REFACTOR] 아이콘 추가 시 patch 버전 업데이트 되도록 수정 [REFACTOR] 아이콘 추가 시 patch 버전 업데이트되도록 수정 Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 아이콘 추가시에 patch 버전 업데이트 되도록 수정

1 participant