Skip to content

Commit bd81d90

Browse files
ci: exclude top-level scripts/prod files from hybrid system test
The hybrid_system_test path-trigger exclude glob was "scripts/prod/**/*", which under fnmatch requires a subdirectory and so did NOT match top-level files like scripts/prod/common_lib.py. As a result, prod-script-only changes still triggered the (irrelevant, infra-heavy) hybrid system test instead of being skipped. Use "scripts/prod/**" so all files under scripts/prod are excluded, matching intent. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent de95d91 commit bd81d90

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/hybrid_system_test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ env:
1616
cluster_name: hybrid-system-test
1717
crate_triggers: "apollo_node,apollo_deployments,apollo_integration_tests"
1818
path_triggers: ".github/workflows/hybrid_system_test.yaml,scripts/**,deployments/sequencer/**,deployments/images/**"
19-
path_triggers_exclude: "scripts/prod/**/*"
19+
path_triggers_exclude: "scripts/prod/**"
2020
pvc_storage_class_name: "premium-rwo"
2121
anvil_port: "8545"
2222
dockerfile_base_path: ./deployments/images/sequencer

0 commit comments

Comments
 (0)