Skip to content

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

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)#23
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

Ready to act? Review this PR in Change Stack to turn feedback into patch suggestions you can inspect and refine.

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: 0c66704e-c5a4-41b5-98a2-50c35735b998

📥 Commits

Reviewing files that changed from the base of the PR and between 779cfe1 and 5d9635f.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated internal GitHub automation configuration for pull request validation.

Walkthrough

Adds a Mergify pull_request_rules entry in .github/mergify.yml to validate T-ID format compliance in PR titles and commit messages. The rule flags violations of the T<digits>: <text> pattern (with optional scope: prefix) by applying the invalid-task-id label.

Changes

T-ID Format Validation

Layer / File(s) Summary
Mergify T-ID validation rule
.github/mergify.yml
New pull_request_rules item enforces T<digits>: <text> (optional scope: prefix) format on PR titles and commit first lines, applying invalid-task-id label when format violations are detected.

Possibly related PRs

  • vyos/vyatta-biosdevname#12: Prior PR modifies the same .github/mergify.yml T-ID validation rules; this PR adds the invalid-task-id labeling rule while the prior PR removed local rules in favor of baseline T-ID checks.
✨ 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 20:59
@andamasov andamasov merged commit e8c6d94 into rolling Jun 5, 2026
6 checks passed
@andamasov andamasov deleted the feat/T8966-product-tid branch June 5, 2026 20:59
@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