Skip to content

Commit 2f05875

Browse files
committed
ci: align pre-commit hooks and adopt org-wide GitHub defaults
Adopt the shared pre-commit baseline: conventional-commit message validation (commit-msg stage) and the centralized add-license hook from the org .github repository (v1.0.0 - excludes .github/ templates and never rewrites LICENSE files). Run pre-commit CI only on files modified by the PR, and add the caller stub for the org-wide conventional-pr reusable workflow (PR title validation + type labeling). PR and issue templates are no longer stored per repository: they are inherited from the org-wide defaults in triton-inference-server/.github (issues route to the server repository). TRI-1100
1 parent 1826863 commit 2f05875

6 files changed

Lines changed: 45 additions & 94 deletions

File tree

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 0 additions & 5 deletions
This file was deleted.

.github/PULL_REQUEST_TEMPLATE/pull_request_template_external_contrib.md

Lines changed: 0 additions & 36 deletions
This file was deleted.

.github/PULL_REQUEST_TEMPLATE/pull_request_template_internal_contrib.md

Lines changed: 0 additions & 36 deletions
This file was deleted.

.github/pull_request_template.md

Lines changed: 0 additions & 13 deletions
This file was deleted.
Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
# Copyright 2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2+
#
3+
# Redistribution and use in source and binary forms, with or without
4+
# modification, are permitted provided that the following conditions
5+
# are met:
6+
# * Redistributions of source code must retain the above copyright
7+
# notice, this list of conditions and the following disclaimer.
8+
# * Redistributions in binary form must reproduce the above copyright
9+
# notice, this list of conditions and the following disclaimer in the
10+
# documentation and/or other materials provided with the distribution.
11+
# * Neither the name of NVIDIA CORPORATION nor the names of its
12+
# contributors may be used to endorse or promote products derived
13+
# from this software without specific prior written permission.
14+
#
15+
# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS ``AS IS'' AND ANY
16+
# EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
17+
# IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
18+
# PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
19+
# CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
20+
# EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
21+
# PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
22+
# PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
23+
# OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
24+
# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
25+
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
26+
27+
# Thin caller: the logic lives in the org-wide reusable workflow, pinned by
28+
# tag. Workflow changes ship as a new tag in the .github repository plus a
29+
# one-line rev bump here (tags are write-once, never moved).
30+
31+
name: conventional-pr
32+
33+
on:
34+
pull_request:
35+
types: [opened, edited, synchronize, reopened]
36+
37+
jobs:
38+
conventional-pr:
39+
permissions:
40+
pull-requests: write
41+
uses: triton-inference-server/.github/.github/workflows/conventional-pr.yml@v1.0.0

.pre-commit-config.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@ repos:
5757
args: ["--toml", "pyproject.toml"]
5858
exclude: (?x)^(.*stemmer.*|.*stop_words.*|^CHANGELOG.md$)
5959
# Validates commit messages against the Conventional Commits format
60-
# (<commit_type>: <title>). Replaces the manual commit-type checklist that
61-
# used to live in the PR template.
60+
# (<commit_type>: <title>); PR titles are validated org-wide by the
61+
# conventional-pr reusable workflow.
6262
- repo: https://github.com/compilerla/conventional-pre-commit
6363
rev: v4.4.0
6464
hooks:
@@ -83,7 +83,7 @@ repos:
8383
- id: requirements-txt-fixer
8484
- id: trailing-whitespace
8585

86-
- repo: https://github.com/triton-inference-server/developer_tools
87-
rev: v0.2.1
86+
- repo: https://github.com/triton-inference-server/.github
87+
rev: v1.0.0
8888
hooks:
8989
- id: add-license

0 commit comments

Comments
 (0)