diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-08-05 00:54:46 +0300 |
|---|---|---|
| committer | Yuriy Andamasov <yuriy@vyos.io> | 2026-08-05 00:54:46 +0300 |
| commit | 8982245ff77cc66be642cdcb5017b31147e51e69 (patch) | |
| tree | d3dbda3f2e078a0174cfcd8a9cdcac001deb11b4 | |
| parent | 6dea44971db1addc73682f3a82add133bcaee2de (diff) | |
| download | vyos-build-nos-task-id.tar.gz vyos-build-nos-task-id.zip | |
T9164: accept NOS-/VD- Jira keys in task-id check; pin NOS in coderabbitnos-task-id
VD tracker project renamed to NOS (2026-07) - PR titles and commit headlines
using the new NOS- key must stop being flagged invalid-task-id; legacy VD-
keys stay accepted, and CodeRabbit now resolves NOS alongside VD. Fleet
replication of canary vyos/vyos-1x#5379, whose adversarial review tightened
the regexes to ASCII [0-9] + literal-space separators.
🤖 Generated by [robots](https://vyos.io)
| -rw-r--r-- | .coderabbit.yaml | 1 | ||||
| -rw-r--r-- | .github/mergify.yml | 14 |
2 files changed, 9 insertions, 6 deletions
diff --git a/.coderabbit.yaml b/.coderabbit.yaml index 6b27c5d4..f7cc930e 100644 --- a/.coderabbit.yaml +++ b/.coderabbit.yaml @@ -26,4 +26,5 @@ knowledge_base: # source where it is not. usage: auto project_keys: + - NOS - VD diff --git a/.github/mergify.yml b/.github/mergify.yml index 54a485dc..2d79068a 100644 --- a/.github/mergify.yml +++ b/.github/mergify.yml @@ -33,10 +33,12 @@ pull_request_rules: - name: Flag product T-ID format violation in PR title or commit messages description: > Product-repo convention: PR title and every commit's first line must - match `T<digits>: <text>` (optional `scope: ` prefix). Relocated from - the central config (T8966) so the T-ID convention is opt-in per product - repo. Name is intentionally distinct from any central rule name so this - stays additive (not an `extends:` override). + match a `T<digits>:`, `NOS-<digits>:` or legacy `VD-<digits>:` task key + followed by text (optional `scope: ` prefix). NOS is the renamed VD Jira + project (2026-07). Relocated from the central config (T8966) so the T-ID + convention is opt-in per product repo. Name is intentionally distinct + from any central rule name so this stays additive (not an `extends:` + override). conditions: - '-closed' - '-merged' @@ -44,10 +46,10 @@ pull_request_rules: - 'author!=copilot-swe-agent' - 'author!=vyosbot' - or: - - '-title~=^(([a-zA-Z0-9\-_.]+:\s)?)T\d+:\s+[^\s]+.*' + - '-title~=^(([a-zA-Z0-9\-_.]+:[ ])?)(T[0-9]+|NOS-[0-9]+|VD-[0-9]+):[ ]+[^\s]+.*' - and: - 'label!=legacy' - - 'commits[*].commit_message~=^(?!(([a-zA-Z0-9\-_.]+:\s)?)T\d+:\s+[^\s]+).*' + - 'commits[*].commit_message~=^(?!(([a-zA-Z0-9\-_.]+:[ ])?)(T[0-9]+|NOS-[0-9]+|VD-[0-9]+):[ ]+[^\s]+).*' actions: label: toggle: |