Skip to content

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

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#1218
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: 91dc0ac8-2e7c-4705-9443-460ea4b107b5

📥 Commits

Reviewing files that changed from the base of the PR and between 4a34962 and 481e11b.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated pull request validation rules to exclude additional bot-authored contributions from specific label checks.

Walkthrough

Updated Mergify rule for T-ID format validation to exclude bot-authored PRs. Added three author-based conditions to the invalid-task-id label rule: generic bot suffix pattern ([bot]), copilot-swe-agent, and vyosbot.

Changes

Mergify T-ID validation bot author exclusion

Layer / File(s) Summary
Bot author exclusion conditions
.github/mergify.yml
Lines 43–45 add three author conditions to the invalid-task-id rule, excluding generic bots matching [bot] suffix, copilot-swe-agent, and vyosbot from triggering the T-ID format validation label.

Possibly related PRs

  • vyos/vyos-build#1217: Introduces the T-ID format validation rule that this PR refines with additional bot-author exclusions.
  • vyos/vyos-build#1188: Also modifies Mergify T-ID validation behavior in .github/mergify.yml.
✨ 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 2f44024 into rolling Jun 6, 2026
8 of 9 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 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

Development

Successfully merging this pull request may close these issues.

1 participant