Skip to content

fix: remove duplicate labels used in repo-settings target#21

Open
yocaba wants to merge 1 commit into
mainfrom
fix/labels
Open

fix: remove duplicate labels used in repo-settings target#21
yocaba wants to merge 1 commit into
mainfrom
fix/labels

Conversation

@yocaba

@yocaba yocaba commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

What

Remove duplicate labels used in repo-settings target.

Why

help wanted vs. help-wanted
good first issue vs. good-first-issue
enhancement vs. feature

Since we only add labels, I went with the GitHub defaults for all three cases.

Testing

none

Checklist

  • Tests added/updated
  • No breaking changes (or upgrade path documented above)
  • Readable commit history (squashed and cleaned up as desired)
  • AI code review considered and comments resolved

Summary by CodeRabbit

  • Chores
    • Updated the repository’s synced label list to remove a few labels from the default set.
    • Existing labels such as go, ok-to-helm, dependencies, and github-actions remain available.

@yocaba yocaba enabled auto-merge July 3, 2026 16:10
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The REPO_LABELS variable in common.mk was modified to remove three label entries (feature, help-wanted, good-first-issue) from the label list used by the repo-settings target, while retaining go, ok-to-helm, dependencies, and github-actions.

Changes

REPO_LABELS Update

Layer / File(s) Summary
Trim label list
common.mk
Removed feature, help-wanted, and good-first-issue entries from the REPO_LABELS variable used for syncing GitHub labels.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main change: removing duplicate labels from the repo-settings target.
Description check ✅ Passed The description covers What, Why, Testing, and Checklist sections and is mostly complete.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/labels

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.

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.

2 participants