[ENH] Enable check-yaml in CI (#2078)

Exclude Helm templates from YAML linting as they are not valid YAML Fixes #2053
chroma-core · Apr 30, 2024 · ff93ecd · ff93ecd
1 parent 4b2599c
commit ff93ecd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/.github/workflows/chroma-lint.yml b/.github/workflows/chroma-lint.yml
@@ -25,3 +25,4 @@ jobs:
           pre-commit run --all-files black
           pre-commit run --all-files flake8
           pre-commit run --all-files prettier
+          pre-commit run --all-files check-yaml
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -10,6 +10,8 @@ repos:
       - id: requirements-txt-fixer
       - id: check-yaml
         args: ["--allow-multiple-documents"]
+        # Exclude Helm templates from YAML linting as they are not valid YAML
+        exclude: "k8s/distributed-chroma/templates/"
       - id: check-xml
       - id: check-merge-conflict
       - id: check-case-conflict