Skip to content

feat(review): skip auto-review when a configured label is present via review.auto_review.skip_labels #2062

Description

@JSONbored

Part of #1954. Add review.auto_review.skip_labels (list, e.g. wip, do-not-review) so a PR carrying any configured label is skipped by the review path. Deterministic label-name match; default empty ⇒ no skips. Complements ignore_title_keywords for teams that gate via labels.

Deliverables

  • Parse skip_labels as a bounded, deduped, lowercased string list in parseReviewConfig (src/signals/focus-manifest.ts:653); serialize in reviewConfigToJson
  • Add a label branch to decideReviewEligibility (src/review/review-eligibility.ts): skipReason="skip_label" when any PR label matches (case-insensitive)
  • Thread the PR's current labels into the eligibility call at maybePublishPrPublicSurface (src/queue/processors.ts:4533)
  • Unit tests: matching label ⇒ skip, no match ⇒ eligible, empty list ⇒ eligible, case-insensitive match, no-labels PR; 97% branch

References

  • src/queue/processors.ts:4533
  • src/review/review-eligibility.ts
  • src/signals/focus-manifest.ts:653
  • #1954

Part of #1954.


size: S · gittensor:feature

Metadata

Metadata

Assignees

No one assigned

    Labels

    gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.help wantedExtra attention is needed

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions