From bfc3faff2f1dc8d7827c474a3bb61578b233d0c8 Mon Sep 17 00:00:00 2001 From: Ramakrishna Prabhu Date: Thu, 25 Jun 2026 14:39:13 -0500 Subject: [PATCH 1/3] chore(ci): exclude skill-card.md from trailing-whitespace hook NVSkills-Eval regenerates skill-card.md with trailing whitespace after SKILL.md changes; exclude it alongside skill.oms.sig. Co-Authored-By: Claude Sonnet 4.6 Signed-off-by: Ramakrishna Prabhu --- .pre-commit-config.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 5ad3b356a1..4c87061de9 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -8,7 +8,7 @@ repos: - id: end-of-file-fixer exclude: ^(datasets|helmchart)/.*\.(mps|json|yaml|yml|txt)$|^skills/.*/skill\.oms\.sig$ - id: trailing-whitespace - exclude: ^datasets/.*\.(mps|json|yaml|yml|txt)$|^skills/.*/skill\.oms\.sig$ + exclude: ^datasets/.*\.(mps|json|yaml|yml|txt)$|^skills/.*/skill\.oms\.sig$|^skills/.*/skill-card\.md$|^skills/.*/BENCHMARK\.md$ - id: check-builtin-literals - id: check-executables-have-shebangs - id: check-json From ff7d038b7bc6c51cfe7784ac7749188b63c82a52 Mon Sep 17 00:00:00 2001 From: Ramakrishna Prabhu Date: Thu, 25 Jun 2026 15:00:22 -0500 Subject: [PATCH 2/3] chore(ci): exclude NVSkills-generated files from trailing-whitespace hook NVSkills-Eval regenerates skill-card.md and BENCHMARK.md with trailing whitespace after SKILL.md changes; exclude them alongside skill.oms.sig. Use verbose (?x) regex format for readability. Co-Authored-By: Claude Sonnet 4.6 Signed-off-by: Ramakrishna Prabhu --- .pre-commit-config.yaml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 4c87061de9..219cfcb402 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -8,7 +8,13 @@ repos: - id: end-of-file-fixer exclude: ^(datasets|helmchart)/.*\.(mps|json|yaml|yml|txt)$|^skills/.*/skill\.oms\.sig$ - id: trailing-whitespace - exclude: ^datasets/.*\.(mps|json|yaml|yml|txt)$|^skills/.*/skill\.oms\.sig$|^skills/.*/skill-card\.md$|^skills/.*/BENCHMARK\.md$ + exclude: | + (?x)^( + ^datasets/.*\.(mps|json|yaml|yml|txt)$| + ^skills/.*/skill\.oms\.sig$| + ^skills/.*/skill-card\.md$| + ^skills/.*/BENCHMARK\.md$ + ) - id: check-builtin-literals - id: check-executables-have-shebangs - id: check-json From 626ec5afde0236323c5cf48483453a9513ca353d Mon Sep 17 00:00:00 2001 From: Ramakrishna Prabhu Date: Thu, 25 Jun 2026 15:04:17 -0500 Subject: [PATCH 3/3] chore(ci): exclude NVSkills-generated files at top-level pre-commit exclude skill.oms.sig, skill-card.md, and BENCHMARK.md are produced by an internal NVSkills-Eval service alongside a content-based signature; exclude them from all hooks at the top level rather than per-hook. Co-Authored-By: Claude Sonnet 4.6 Signed-off-by: Ramakrishna Prabhu --- .pre-commit-config.yaml | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 219cfcb402..f6ccc04034 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,20 +1,21 @@ # SPDX-FileCopyrightText: Copyright (c) 2019-2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved. # SPDX-License-Identifier: Apache-2.0 +exclude: | + (?x)^( + skills/.*/skill\.oms\.sig| + skills/.*/skill-card\.md| + skills/.*/BENCHMARK\.md + )$ + repos: - repo: https://github.com/pre-commit/pre-commit-hooks rev: 'v6.0.0' hooks: - id: end-of-file-fixer - exclude: ^(datasets|helmchart)/.*\.(mps|json|yaml|yml|txt)$|^skills/.*/skill\.oms\.sig$ + exclude: ^(datasets|helmchart)/.*\.(mps|json|yaml|yml|txt)$ - id: trailing-whitespace - exclude: | - (?x)^( - ^datasets/.*\.(mps|json|yaml|yml|txt)$| - ^skills/.*/skill\.oms\.sig$| - ^skills/.*/skill-card\.md$| - ^skills/.*/BENCHMARK\.md$ - ) + exclude: ^datasets/.*\.(mps|json|yaml|yml|txt)$ - id: check-builtin-literals - id: check-executables-have-shebangs - id: check-json