Chore(project): 이슈 및 PR 작성 시 Assignees 자동 할당#298
Merged
jin-evergreen merged 1 commit intodevelopfrom Mar 4, 2026
Merged
Conversation
📝 WalkthroughSummary by CodeRabbit새로운 기능
Walkthrough이슈/PR이 생성될 때 레이블 추가와 함께 작성자에게 자동으로 할당하는 기능을 추가했어요. 기존 레이블 처리 플로우 이후에 assign 로직이 실행되며, 에러 발생 시에도 워크플로우 실패는 방지하는 구조입니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/assign-auto-label.yml:
- Around line 78-86: The empty catch swallowing errors around the
github.rest.issues.addAssignees call hides failures; update the catch block that
wraps github.rest.issues.addAssignees (the try/catch surrounding issueNumber and
assignees: [author]) to log a warning including the caught error and context
(e.g., issueNumber and author) instead of doing nothing—use the Action logger
(core.warning or console.warn) to emit a clear message like "Failed to add
assignee" plus the error details so failures remain non-fatal but are
discoverable.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 132eec50-ff4b-42fc-ad7e-413fe47c861c
📒 Files selected for processing (1)
.github/workflows/assign-auto-label.yml
Comment on lines
+78
to
+86
| try { | ||
| await github.rest.issues.addAssignees({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: issueNumber, | ||
| assignees: [author] | ||
| }); | ||
| } catch (error) { | ||
| } |
There was a problem hiding this comment.
Must: 자동 할당 실패를 조용히 무시하지 않도록 로그를 남겨 주세요.
Line 85-86에서 예외를 비워두면 할당 실패 시 원인 추적이 어려워집니다. 워크플로우를 실패시키지 않는 현재 방향은 유지하되, 최소한 경고 로그는 남겨 주시는 것이 좋습니다.
제안 패치
try {
await github.rest.issues.addAssignees({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: issueNumber,
assignees: [author]
});
} catch (error) {
+ console.warn(
+ `[auto-labeler] failed to assign "${author}" to #${issueNumber}: ${error?.message || error}`
+ );
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| try { | |
| await github.rest.issues.addAssignees({ | |
| owner: context.repo.owner, | |
| repo: context.repo.repo, | |
| issue_number: issueNumber, | |
| assignees: [author] | |
| }); | |
| } catch (error) { | |
| } | |
| try { | |
| await github.rest.issues.addAssignees({ | |
| owner: context.repo.owner, | |
| repo: context.repo.repo, | |
| issue_number: issueNumber, | |
| assignees: [author] | |
| }); | |
| } catch (error) { | |
| console.warn( | |
| `[auto-labeler] failed to assign "${author}" to #${issueNumber}: ${error?.message || error}` | |
| ); | |
| } |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.github/workflows/assign-auto-label.yml around lines 78 - 86, The empty
catch swallowing errors around the github.rest.issues.addAssignees call hides
failures; update the catch block that wraps github.rest.issues.addAssignees (the
try/catch surrounding issueNumber and assignees: [author]) to log a warning
including the caught error and context (e.g., issueNumber and author) instead of
doing nothing—use the Action logger (core.warning or console.warn) to emit a
clear message like "Failed to add assignee" plus the error details so failures
remain non-fatal but are discoverable.
Sohyunnnn
approved these changes
Mar 4, 2026
Contributor
Sohyunnnn
left a comment
Sohyunnnn
left a comment
There was a problem hiding this comment.
👍👍👍 진석님 덕분에 DX가 한층 편해지는 거 같아요 감사합니다 !!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Summary
이슈 및 PR 작성 시 라벨 뿐 아니라 Assignees도 자동 할당되도록 했습니다.
📚 Tasks
assign-auto-label.yml파일 수정🔍 Describe
이슈 및 PR 작성 시 Assignees 자동 할당
기존에는 라벨만 자동 할당되도록 설정했는데, Assignees도 함께 자동으로 할당되도록 관련 로직을 추가했습니다.
👀 To Reviewer