.pre-commit-config.yaml 3.21 KB
Newer Older
1
# SPDX-FileCopyrightText: Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2
3
4
5
6
7
8
9
10
11
12
13
14
15
# SPDX-License-Identifier: Apache-2.0
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

16
exclude: ^(src/grpc_generated|.*\.patch$|.*/connect/.*\.py)
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
repos:
- repo: https://github.com/timothycrosley/isort
  rev: 5.12.0
  hooks:
  - id: isort
    additional_dependencies: [toml]
- repo: https://github.com/psf/black
  rev: 23.1.0
  hooks:
  - id: black
    types_or: [python, cython]
- repo: https://github.com/PyCQA/flake8
  rev: 5.0.4
  hooks:
  - id: flake8
    args: [--max-line-length=88, --select=C,E,F,W,B,B950, --extend-ignore = E203,E501]
    types_or: [python, cython]
- repo: https://github.com/pre-commit/mirrors-clang-format
  rev: v16.0.5
  hooks:
  - id: clang-format
    types_or: [c, c++, cuda, proto, textproto, java]
    args: ["-fallback-style=none", "-style=file", "-i"]
- repo: https://github.com/codespell-project/codespell
  rev: v2.2.4
  hooks:
  - id: codespell
44
    # Use pyproject.toml for codespell configuration
45
    additional_dependencies: [tomli]
Neelay Shah's avatar
Neelay Shah committed
46

47
48
49
50
51
52
53
54
55
56
57
58
# More details about these pre-commit hooks here:
# https://pre-commit.com/hooks.html
- repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v4.4.0
  hooks:
  - id: check-case-conflict
  - id: check-executables-have-shebangs
  - id: check-merge-conflict
  - id: check-json
    exclude: .devcontainer
  - id: check-toml
  - id: check-yaml
59
    exclude: ^.*/templates/.*\.yaml$ #ignore all yaml files in helm chart templates
60
    args: ['--allow-multiple-documents']
61
62
63
64
65
66
  - id: check-shebang-scripts-are-executable
  - id: end-of-file-fixer
    types_or: [c, c++, cuda, proto, textproto, java, python]
  - id: mixed-line-ending
  - id: requirements-txt-fixer
  - id: trailing-whitespace
67
    exclude: lib/llm/tests/data/deepseek-v3.2/.*\.txt$
68

Neelay Shah's avatar
Neelay Shah committed
69
70
71
72
73
74
75
76
# NOTE: removing from pre commit
# will move to gitlab ci to run in proper
# container
#- repo: https://github.com/pre-commit/mirrors-mypy
#  rev: v1.13.0
#  hooks:
#  - id: mypy
#    exclude: model.py # WAR errors about 'model.py' duplicate module name
77
78
79
80
81

# Fast linting
- repo: https://github.com/astral-sh/ruff-pre-commit
  rev: v0.5.2
  hooks:
J Wyman's avatar
J Wyman committed
82
83
  - id: ruff
    args: [--fix, --exit-non-zero-on-fix]
84
85
86

# NOTE: pyright may be able to find other classes of errors not covered above,
# but would require some configuring and venv setup to properly eliminate noise
87
# and give it visiblity into all the local and third_party packages expected.
88
89
90
91
92
93
94
95
96
97
98
99
100
- repo: local
  hooks:
    - id: pytest-marker-report
      name: Report pytest markers (static + inherited)
      entry: python3 scripts/report_pytest_markers.py
      language: python
      pass_filenames: false
      additional_dependencies:
        - pytest
        - tomli
        - pydantic
        - filelock
        - pyyaml