Skip to content

fix: pr-autolabel 타입 미검출 시 기존 라벨 보존 - #90

Merged
6-keem merged 1 commit into
mainfrom
fix/autolabel-no-type
Jun 13, 2026
Merged

fix: pr-autolabel 타입 미검출 시 기존 라벨 보존#90
6-keem merged 1 commit into
mainfrom
fix/autolabel-no-type

Conversation

@6-keem

@6-keem 6-keem commented Jun 13, 2026

Copy link
Copy Markdown
Member

Closes #89

변경 내용

PR 제목에 Conventional 타입 prefix가 없을 때 기존 타입 라벨이 삭제되던 버그 수정 — if (!target) return을 제거 루프 앞으로 이동해 라벨을 보존.

CodeRabbit #86 리뷰 반영.

변경 유형

  • 🐛 버그 수정

체크리스트

Summary by CodeRabbit

Chores

  • PR 자동 라벨 지정 워크플로우의 라벨 동기화 로직을 개선하여 중복 라벨 추가를 방지하고 기존 라벨 보존을 강화했습니다.

early-return을 제거 루프 앞으로 이동. CodeRabbit 리뷰 반영.

Closes #89
@github-actions github-actions Bot added the bug 정상 동작하지 않는 문제 label Jun 13, 2026
@coderabbitai

coderabbitai Bot commented Jun 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 63b6f637-4f61-4af9-b896-c23b6385d9a5

📥 Commits

Reviewing files that changed from the base of the PR and between 6cb2925 and bb0adc9.

📒 Files selected for processing (1)
  • .github/workflows/pr-autolabel.yml

📝 Walkthrough

Walkthrough

PR 제목의 Conventional Commits prefix로부터 타입 라벨을 매핑할 수 없는 경우를 처리하는 가드 로직을 추가합니다. target 라벨이 없을 때 조기 반환하여 기존 라벨이 의도치 않게 삭제되는 버그를 방지합니다.

Changes

PR 자동라벨 워크플로우 수정

Layer / File(s) Summary
타입 미검출 시 early-return 가드
.github/workflows/pr-autolabel.yml
PR 제목에서 매핑되는 타입 라벨(target)을 찾지 못한 경우, 로그를 남기고 즉시 반환하도록 가드 구문을 추가하여 이후 라벨 제거 루프의 실행을 방지합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3분

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 'pr-autolabel 타입 미검출 시 기존 라벨 보존'으로 변경 내용을 명확하게 요약하고 있으며, 이는 라벨 제거 로직 재배치를 통한 버그 수정이라는 핵심 변경 사항과 직접적으로 관련되어 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경 사항은 issue #89의 목표인 'early-return을 라벨 제거 루프 앞으로 이동시켜 타입 미검출 시 기존 타입 라벨 보존'을 완벽하게 구현하고 있습니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항은 pr-autolabel 워크플로우의 라벨 제거 로직 재배치에만 집중되어 있으며, issue #89의 범위 내에서 필요한 수정만 포함하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/autolabel-no-type

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

@6-keem
6-keem merged commit ee5f445 into main Jun 13, 2026
7 of 8 checks passed
@6-keem
6-keem deleted the fix/autolabel-no-type branch June 13, 2026 17:07
@dotbugi dotbugi Bot mentioned this pull request Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 정상 동작하지 않는 문제

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pr-autolabel: 타입 미검출 시 기존 타입 라벨이 삭제되는 버그

1 participant