Skip to content

fix: address CodeRabbit review findings - #7

Merged
tarun-etikala merged 1 commit into
mainfrom
feat/automated-pr-labeling
Apr 23, 2026
Merged

tarun-etikala merged 1 commit into
mainfrom
feat/automated-pr-labeling

Conversation

@tarun-etikala

Copy link
Copy Markdown
Owner
  • Fix image exclusion to match nested directories at any depth (agents//images/, etc.) instead of only top-level images/*
  • Only catch 404 in ensure_label_exists, re-raise 403/429/500
  • Pin PyGithub==2.6.0 to avoid supply-chain risk in pull_request_target
  • Add warning step when size labeling fails for better observability

Description

Jira Ticket

Testing

  • make test passes (run from the affected agent directory)
  • Manual testing performed (describe steps below)
  • No testing required (documentation/config change only)

Checklist

  • I have read CONTRIBUTING.md
  • No .env or secret files are included in this PR
  • All changes are within scope of the linked Jira ticket (if not, explain in Description)

Review Guidance

Related PRs

- Fix image exclusion to match nested directories at any depth
  (agents/*/images/*, etc.) instead of only top-level images/*
- Only catch 404 in ensure_label_exists, re-raise 403/429/500
- Pin PyGithub==2.6.0 to avoid supply-chain risk in pull_request_target
- Add warning step when size labeling fails for better observability

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tarun-etikala
tarun-etikala merged commit efbdce3 into main Apr 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant