Unverified Commit e2371a48 authored by Alec's avatar Alec Committed by GitHub
Browse files

fix(ci): move negation patterns after positives in filters.yaml (#8123)


Co-authored-by: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: default avatarSaravana Periyasamy <saperiyasamy@nvidia.com>
parent a7188f61
...@@ -71,10 +71,6 @@ ci: &ci ...@@ -71,10 +71,6 @@ ci: &ci
- 'container/Dockerfile.test' - 'container/Dockerfile.test'
core: core:
# Exclude documentation files first, then include patterns override for specific paths
- '!**/*.md'
- '!**/*.rst'
- '!**/*.txt'
- *ci - *ci
- 'container/render.py' - 'container/render.py'
- 'container/Dockerfile.template' - 'container/Dockerfile.template'
...@@ -98,6 +94,12 @@ core: ...@@ -98,6 +94,12 @@ core:
- '*.py' - '*.py'
- '*.rs' - '*.rs'
- 'scripts/report_pytest_markers.py' - 'scripts/report_pytest_markers.py'
# Negation patterns MUST come after positive patterns — micromatch processes
# patterns left-to-right, so negations only filter matches accumulated by
# preceding patterns. Placing them first has no effect.
- '!**/*.md'
- '!**/*.rst'
- '!**/*.txt'
operator: operator:
- *ci - *ci
...@@ -108,13 +110,13 @@ operator: ...@@ -108,13 +110,13 @@ operator:
- 'components/src/dynamo/profiler/utils/dgdr_*' - 'components/src/dynamo/profiler/utils/dgdr_*'
deploy: deploy:
- '!**/*.md'
- '!**/*.rst'
- *ci - *ci
- 'deploy/helm/**' - 'deploy/helm/**'
- 'deploy/utils/**' - 'deploy/utils/**'
- 'deploy/snapshot/**' - 'deploy/snapshot/**'
- 'tests/deploy/**' - 'tests/deploy/**'
- '!**/*.md'
- '!**/*.rst'
planner: planner:
- 'container/templates/planner.Dockerfile' - 'container/templates/planner.Dockerfile'
...@@ -124,33 +126,31 @@ planner: ...@@ -124,33 +126,31 @@ planner:
- 'components/src/dynamo/global_router/**' - 'components/src/dynamo/global_router/**'
vllm: vllm:
- '!**/*.md'
- '!**/*.rst'
- 'container/deps/requirements.vllm.txt' - 'container/deps/requirements.vllm.txt'
- 'container/deps/vllm/**' - 'container/deps/vllm/**'
- 'examples/backends/vllm/**' - 'examples/backends/vllm/**'
- 'components/src/dynamo/vllm/**' - 'components/src/dynamo/vllm/**'
- 'container/templates/vllm_*' - 'container/templates/vllm_*'
sglang:
- '!**/*.md' - '!**/*.md'
- '!**/*.rst' - '!**/*.rst'
sglang:
- 'examples/backends/sglang/**' - 'examples/backends/sglang/**'
- 'components/src/dynamo/sglang/**' - 'components/src/dynamo/sglang/**'
- 'container/templates/sglang_*' - 'container/templates/sglang_*'
trtllm:
- '!**/*.md' - '!**/*.md'
- '!**/*.rst' - '!**/*.rst'
trtllm:
- 'container/deps/trtllm/**' - 'container/deps/trtllm/**'
- 'examples/backends/trtllm/**' - 'examples/backends/trtllm/**'
- 'components/src/dynamo/trtllm/**' - 'components/src/dynamo/trtllm/**'
- 'container/build_trtllm_wheel.sh' - 'container/build_trtllm_wheel.sh'
- 'container/templates/trtllm_*' - 'container/templates/trtllm_*'
frontend:
- '!**/*.md' - '!**/*.md'
- '!**/*.rst' - '!**/*.rst'
frontend:
- *ci - *ci
- '.cargo/config.toml' - '.cargo/config.toml'
- 'lib/**' - 'lib/**'
...@@ -164,6 +164,8 @@ frontend: ...@@ -164,6 +164,8 @@ frontend:
- 'components/src/dynamo/common/**' - 'components/src/dynamo/common/**'
- 'deploy/inference-gateway/**' - 'deploy/inference-gateway/**'
- 'container/templates/frontend.Dockerfile' - 'container/templates/frontend.Dockerfile'
- '!**/*.md'
- '!**/*.rst'
rust: rust:
- '.github/workflows/pre-merge.yml' - '.github/workflows/pre-merge.yml'
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment