Skip to content

ci: T8966: add product T-ID title/commit check (opt-in relocation)#110

Merged
andamasov merged 1 commit into
rollingfrom
feat/T8966-product-tid
Jun 5, 2026
Merged

ci: T8966: add product T-ID title/commit check (opt-in relocation)#110
andamasov merged 1 commit into
rollingfrom
feat/T8966-product-tid

Conversation

@andamasov

Copy link
Copy Markdown
Member

Relocates the central strict T-ID title/commit check into this product repo as an opt-in rule (toggles invalid-task-id), per T8966. Additive — does not change any existing rule.

🤖 Generated by robots

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Complex PR? Review this PR in Change Stack to move by importance, not file order.

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: 540e1828-6b8e-476b-bf15-06c645b4b2ea

📥 Commits

Reviewing files that changed from the base of the PR and between db96db5 and 5611bb0.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated pull request validation rules to enforce consistent task ID formatting in PR titles and commit messages, automatically labeling non-compliant submissions.

Walkthrough

Mergify rule added to validate PR titles and commit first-line format against T-ID convention. When a PR closes or merges with a title or commit message failing the required [scope:]T<digits>: <text> pattern, the invalid-task-id label is toggled.

Changes

Mergify Configuration

Layer / File(s) Summary
T-ID format validation rule
.github/mergify.yml
New pull_request_rules block containing regex conditions that check PR title and commit first line for optional scope: prefix followed by T<digits>: <text>. Toggles invalid-task-id label when format is violated on closed/merged PRs.

Possibly related PRs

  • vyos/vyos-cloud-init#101: Both PRs modify .github/mergify.yml's pull_request_rules for T-ID/title/commit-message validation.
✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch feat/T8966-product-tid

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

@mergify mergify Bot added the rolling label Jun 5, 2026
@andamasov andamasov marked this pull request as ready for review June 5, 2026 21:00
@andamasov andamasov merged commit e4b03ea into rolling Jun 5, 2026
6 of 7 checks passed
@andamasov andamasov deleted the feat/T8966-product-tid branch June 5, 2026 21: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 5, 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