Skip to content

ci: T8966: exempt bots from product T-ID (invalid-task-id) gate#24

Merged
andamasov merged 1 commit into
rollingfrom
feat/T8966-botexcl-tid
Jun 6, 2026
Merged

ci: T8966: exempt bots from product T-ID (invalid-task-id) gate#24
andamasov merged 1 commit into
rollingfrom
feat/T8966-botexcl-tid

Conversation

@andamasov

Copy link
Copy Markdown
Member

Adds the blessed 3-condition bot exclusion to this repo's product T-ID rule so bot PRs aren't gated by the T-ID convention, per T8966. Uniform across all product repos.

🤖 Generated by robots

@coderabbitai

coderabbitai Bot commented Jun 6, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 94fc7ef2-bf2a-4f0c-97c5-9f0fcf00a152

📥 Commits

Reviewing files that changed from the base of the PR and between e8c6d94 and 4b54b84.

📒 Files selected for processing (1)
  • .github/mergify.yml

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated pull request validation automation configuration rules.

Walkthrough

Updated .github/mergify.yml to exclude PRs authored by bots (-author=[bot]), copilot-swe-agent, and vyosbot from the T-ID format violation rule. These automation accounts previously triggered invalid task ID labels on non-T-ID-formatted PRs.

Changes

Mergify Automation Exclusion

Layer / File(s) Summary
T-ID validation rule automation exclusions
.github/mergify.yml
The Mergify rule at lines 35–37 adds conditions to exclude bot authors and two named service accounts (copilot-swe-agent, vyosbot) from triggering the invalid-task-id label when PR title or commit message format does not match the T-ID pattern.

Possibly Related PRs

  • vyos/vyatta-biosdevname#23: Both PRs modify the Mergify T-ID format validation rule in .github/mergify.yml; this PR extends it to exclude bot and automation authors.
✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch feat/T8966-botexcl-tid

Comment @coderabbitai help to get the list of available commands and usage tips.

@mergify mergify Bot added the rolling label Jun 6, 2026
@mergify

mergify Bot commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 📃 Configuration Change Requirements

Wonderful, this rule succeeded.

Mergify configuration change

  • check-success = Configuration changed

@andamasov andamasov marked this pull request as ready for review June 6, 2026 16:01
@andamasov andamasov merged commit 5422beb into rolling Jun 6, 2026
7 checks passed
@andamasov andamasov deleted the feat/T8966-botexcl-tid branch June 6, 2026 16:01
@vyos-bot vyos-bot Bot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed This PR has been mirrored successfully and removed mirror-initiated This PR initiated for mirror sync workflow labels Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mirror-completed This PR has been mirrored successfully rolling

Development

Successfully merging this pull request may close these issues.

1 participant