From 3d44fe06e3a82e63b1391424860d76f95bcb97f3 Mon Sep 17 00:00:00 2001 From: Yuriy Andamasov Date: Tue, 12 May 2026 15:43:23 +0300 Subject: ci(mergify): disable inherited T-ID title/commit-message checks in this repo MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The central baseline at vyos/mergify:.mergify.yml enforces a `T: ` (optional `scope: ` prefix) format on PR titles and every commit's first line — a convention from vyos/vyos-1x and sibling code repos where engineering work tracks against a Phorge task. vyos-documentation has never enforced this. Recent merged PRs use conventional-commits style (`fix(ext): …`, `ci(...): …`, `chore(deps): …`) without an upstream T-ID. After the `extends: mergify` adoption in #2005, the inherited rules started labeling every such PR `invalid-title` and `invalid-commit-title` — pure noise for this repo. Override the two rules by name (Mergify `extends:` semantics: same-name rules in the child REPLACE the parent's). Conditions are constructed to never match (`closed and -closed`), so the `toggle` action always falls into the "remove label" branch and also clears the labels from any PR that picked them up since #2005 merged. 🤖 Generated by [robots](https://vyos.io) --- .github/mergify.yml | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/.github/mergify.yml b/.github/mergify.yml index 21c9921c..8dc66929 100644 --- a/.github/mergify.yml +++ b/.github/mergify.yml @@ -17,3 +17,51 @@ # configuration per Mergify's `extends:` semantics. extends: mergify + +pull_request_rules: + # ------------------------------------------------------------------ + # Override: disable T-ID-format checks for PR title and commit messages. + # + # The central baseline in vyos/mergify enforces `T: ` (with + # optional `scope: ` prefix) on both the PR title and every commit's first + # line. That matches the convention in vyos/vyos-1x and other code repos + # where engineering work tracks against a Phorge task. + # + # vyos-documentation does not require a Phorge T-ID per PR — much of the + # work here is conventional-commits style (`fix(ext): …`, `ci(...): …`, + # `chore(deps): …`) with no upstream ticket. Inheriting the central rules + # labels every such PR `invalid-title` / `invalid-commit-title`, which is + # noise. + # + # Per Mergify `extends:` semantics, same-name rules in the child REPLACE + # the parent's. The two rules below keep the original names so the parent + # versions are dropped; conditions are constructed to never match (a PR + # cannot simultaneously be closed and not closed), so the `toggle` action + # always falls into the "remove label" branch — cleaning up the labels on + # any PR that previously had them. + # ------------------------------------------------------------------ + + - name: Flag PR title not matching T-ID format + description: > + Disabled in vyos-documentation — this repo does not require Phorge + T-IDs in PR titles. Conventional-commits style is acceptable. + conditions: + - closed + - '-closed' + actions: + label: + toggle: + - invalid-title + + - name: Flag commit message not matching T-ID format + description: > + Disabled in vyos-documentation — this repo does not require Phorge + T-IDs in commit message headlines. Conventional-commits style is + acceptable. + conditions: + - closed + - '-closed' + actions: + label: + toggle: + - invalid-commit-title -- cgit v1.2.3