Skip to content

T8966: add legacy-label escape to invalid-task-id rule (commit check exempt)#25

Merged
andamasov merged 1 commit into
rollingfrom
feat/T8966-legacy-escape
Jun 7, 2026
Merged

T8966: add legacy-label escape to invalid-task-id rule (commit check exempt)#25
andamasov merged 1 commit into
rollingfrom
feat/T8966-legacy-escape

Conversation

@andamasov

Copy link
Copy Markdown
Member

Adds a maintainer-controlled legacy label escape to the per-repo T-ID rule (toggles invalid-task-id). The PR title check stays enforced for every PR; the per-commit check is exempted when a PR carries the legacy label — an escape hatch for grandfathered PRs whose commit history cannot be rewritten (repos that squash-merge and block force-push). Pilot verified on vyos/vyos.vyos. Change-management: IS-539. Tracked under T8966.

🤖 Generated by robots

@andamasov andamasov merged commit 24ed968 into rolling Jun 7, 2026
0 of 3 checks passed
@andamasov andamasov deleted the feat/T8966-legacy-escape branch June 7, 2026 22:49
@coderabbitai

coderabbitai Bot commented Jun 7, 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: 53308f4c-7419-4866-8bc4-8e85ef2ed301

📥 Commits

Reviewing files that changed from the base of the PR and between 5422beb and 06679a1.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated PR validation configuration to support label-based exemptions for product ID format requirements while maintaining existing commit message formatting checks.

Walkthrough

The Mergify configuration rule that detects product T-ID format violations in commit messages is updated to add a label-based exemption. PRs labeled legacy now bypass the commit-message validation, while the PR title check and the pattern format requirement remain in place.

Changes

Mergify Configuration

Layer / File(s) Summary
T-ID violation rule with label exemption
.github/mergify.yml
The invalid-task-id rule now requires label!=legacy alongside the commit-message pattern mismatch condition, exempting legacy-labeled PRs from the T-ID format check while preserving title validation and commit format enforcement.

Possibly related PRs

✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch feat/T8966-legacy-escape

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

@mergify mergify Bot added the rolling label Jun 7, 2026
@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 7, 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