Skip to content

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

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#43
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: 9567af89-d01a-47ad-acb2-ca47c3df4eba

📥 Commits

Reviewing files that changed from the base of the PR and between b8abb7c and daf2ec2.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated automation rules to exclude certain bot accounts from pull request validation checks.

Walkthrough

Updated Mergify rule to exclude bot accounts and specific agents (copilot-swe-agent, vyosbot) from T-ID format violation checks via three additional author-based conditions in .github/mergify.yml (lines 35–37).

Changes

Mergify T-ID validation rule author exclusions

Layer / File(s) Summary
Author-based exclusion conditions for T-ID validation
.github/mergify.yml
Rule conditions exclude generic bot authors (-author=\[bot\]$), copilot-swe-agent, and vyosbot from triggering T-ID format violation labeling.

Possibly related PRs

  • vyos/ipaddrcheck#42: Both PRs modify .github/mergify.yml for the same product T-ID validation rule (one adds T-ID checks, the other adds author-based exclusions).
  • vyos/ipaddrcheck#27: Both PRs modify Mergify configuration around PR title/commit T-ID validation and related labeling rules.
✨ 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:00
@andamasov
andamasov merged commit 5db51cd into rolling Jun 6, 2026
7 checks passed
@andamasov
andamasov deleted the feat/T8966-botexcl-tid branch June 6, 2026 16:00
@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