Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ LICENSE @NVIDIA/cuvs-docs-codeowners

# CI code owners
/.github/ @NVIDIA/adi-ci-codeowners
/.yamllint.yaml @NVIDIA/adi-ci-codeowners
/ci/ @NVIDIA/adi-ci-codeowners

# packaging code owners
Expand Down
11 changes: 11 additions & 0 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,20 @@ jobs:
files_yaml: |
test_java:
- '**'
- '!.github/CODEOWNERS'

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proposing expanding this list, should help avoid some unnecessary CI runs.

- '!.github/build.yaml'
- '!.github/copy-pr-bot.yaml'
- '!.github/ops-bot.yaml'
- '!.github/release.yml'
- '!.gihtub/test.yaml'
- '!.github/zizmor.yml'
- '!.gitignore'
- '!.pre-commit-config.yaml'
- '!.yamllint.yaml'
- '!README.md'
- '!SECURITY.md'
- '!ci/check_style.sh'
- '!ci/release/update-version.sh'
permissions:
actions: read
contents: read
Expand Down
89 changes: 51 additions & 38 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,41 +2,54 @@
# SPDX-License-Identifier: Apache-2.0

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v6.0.0
hooks:
- id: check-json
- id: trailing-whitespace
- id: end-of-file-fixer
- id: check-symlinks
- id: check-xml
- repo: https://github.com/rapidsai/pre-commit-hooks
rev: v1.5.1
hooks:
- id: verify-copyright
name: verify-copyright
args: [--fix, --spdx]
files: |
(?x)
[.](cmake|cpp|cu|cuh|h|hpp|sh|pxd|py|pyx|rs|java)$|
CMakeLists[.]txt$|
CMakeLists_standalone[.]txt$|
meta[.]yaml$|
dependencies[.]yaml$|
^[.]pre-commit-config[.]yaml$
# TODO: Re-enable once verify-codeowners supports --org parameter
# - id: verify-codeowners
# args: [--fix, --org=NVIDIA, --project-prefix=cuvs-lucene]
- repo: https://github.com/rapidsai/dependency-file-generator
rev: v1.21.0
hooks:
- id: rapids-dependency-file-generator
args: ["--clean", "--warn-all", "--strict"]
- repo: https://github.com/shellcheck-py/shellcheck-py
rev: v0.11.0.1
hooks:
- id: shellcheck
- repo: https://github.com/zizmorcore/zizmor-pre-commit
rev: v1.26.1
hooks:
- id: zizmor
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v6.0.0
hooks:
- id: check-json
- id: trailing-whitespace
- id: end-of-file-fixer
- id: check-symlinks
- id: check-xml
- repo: https://github.com/rapidsai/pre-commit-hooks
rev: v1.6.1
hooks:
- id: verify-copyright
name: verify-copyright
args: [--fix, --spdx]
files: |
(?x)
[.](cmake|cpp|cu|cuh|h|hpp|sh|pxd|py|pyx|rs|java)$|
CMakeLists[.]txt$|
CMakeLists_standalone[.]txt$|
meta[.]yaml$|
dependencies[.]yaml$|
^[.]pre-commit-config[.]yaml$
# TODO: Re-enable once verify-codeowners supports --org parameter
# - id: verify-codeowners
# args: [--fix, --org=NVIDIA, --project-prefix=cuvs-lucene]
- repo: https://github.com/rapidsai/dependency-file-generator
rev: v1.21.0
hooks:
- id: rapids-dependency-file-generator
args: ["--clean", "--warn-all", "--strict"]
- repo: https://github.com/shellcheck-py/shellcheck-py
rev: v0.11.0.1
hooks:
- id: shellcheck
- repo: https://github.com/adrienverge/yamllint
rev: v1.38.0
hooks:
- id: yamllint
additional_dependencies: [pyyaml]
exclude: |
(?x)^(
[.]github/labeler[.]yml$|
conda/environments/.*|
cpp/[.]clang-format$|
cpp/[.]clang-tidy$|
.*xfail\-.*yaml$
)
- repo: https://github.com/zizmorcore/zizmor-pre-commit
rev: v1.28.0
hooks:
- id: zizmor
37 changes: 37 additions & 0 deletions .yamllint.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
# SPDX-FileCopyrightText: Copyright (c) 2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# SPDX-License-Identifier: Apache-2.0
extends: default

rules:
anchors:
forbid-undeclared-aliases: true
forbid-duplicated-anchors: true
forbid-unused-anchors: true
braces:
forbid: false
min-spaces-inside: 0
# allow 1 space for jinja templating in conda recipes
max-spaces-inside: 1
min-spaces-inside-empty: -1
max-spaces-inside-empty: -1
brackets: enable
colons:
max-spaces-before: 0
max-spaces-after: 1
commas:
max-spaces-before: 0
min-spaces-after: 1
max-spaces-after: 1
comments: disable
comments-indentation: disable
document-end: disable
document-start: disable
key-duplicates:
forbid-duplicated-merge-keys: true
line-length: disable
truthy:
allowed-values: ['false', 'true']
# having problematic value in keys is rare... and also
# GitHub Actions' choie of 'on:' triggers this check
# ref: https://github.com/adrienverge/yamllint/issues/430
check-keys: false