Skip to content

fix: skip codeowner notification for kb/review issues#945

Merged
hilram7 merged 1 commit into
devfrom
fix/skip-codeowner-notify-kb-review-issues
May 18, 2026
Merged

fix: skip codeowner notification for kb/review issues#945
hilram7 merged 1 commit into
devfrom
fix/skip-codeowner-notify-kb-review-issues

Conversation

@hilram7
Copy link
Copy Markdown
Collaborator

@hilram7 hilram7 commented May 18, 2026

Summary:

  • KB PR review tracking issues were unintentionally triggering codeowner team notifications via the issue labeler workflow
  • Added an early exit in Step 4 of claude-issue-labeler.yml when a kb/review label is present

Changes:

  • .github/workflows/claude-issue-labeler.yml — added 3-line guard before the codeowner mapping lookup
  • issues with kb/review label now exit Step 4 early with a log message instead of pinging product teams

Testing:

  • Verified logic locally with two bash tests:
    • Labels including kb/review → exits early, no notification sent
    • Labels without kb/review (e.g. auditor, bug) → proceeds to notify codeowners as expected

KB PR review tracking issues were triggering codeowner team mentions
via the issue labeler workflow. Added an early exit in Step 4 when the
kb/review label is present, since these issues are internal tracking
items and product review teams should not be tagged.
@hilram7 hilram7 merged commit 14c9eeb into dev May 18, 2026
7 checks passed
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.

1 participant