Skip to content

Commit d0bc135

Browse files
committed
fix: correct Bazel command syntax for --keep_going flag
The --keep_going flag was placed after the -- separator, causing Bazel to interpret it as a target '//:-keep_going' instead of a command flag. Fixed both Linux and macOS CI jobs: - Move --keep_going before the -- separator - bazel build --keep_going -- [targets...] [exclusions...] This allows CI to continue building other targets even if some fail, ensuring our hermetic Go toolchain validation can complete successfully even if unrelated components have issues.
1 parent cc88f65 commit d0bc135

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ jobs:
103103
run: |
104104
# Build core working targets, exclude incomplete/problematic examples
105105
# Using explicit inclusions to avoid JavaScript component issues
106-
bazel build -- \
106+
bazel build --keep_going -- \
107107
//examples/go_component/... \
108108
//examples/basic/... \
109109
//examples/simple_module/... \
@@ -121,8 +121,7 @@ jobs:
121121
//test/unit/... \
122122
//test/integration/... \
123123
//docs-site/... \
124-
-//examples/cpp_component/multi_component_system:analytics_service \
125-
--keep_going
124+
-//examples/cpp_component/multi_component_system:analytics_service
126125
127126
- name: Run Tests
128127
run: bazel test --test_output=errors -- //test/integration:basic_component_build_test //test/integration:basic_component_validation //test/unit:unit_tests
@@ -187,7 +186,7 @@ jobs:
187186
run: |
188187
# Build core working targets using explicit inclusions
189188
# Same approach as Linux but with macOS-specific exclusions
190-
bazel build -- \
189+
bazel build --keep_going -- \
191190
//examples/go_component/... \
192191
//examples/basic/... \
193192
//examples/simple_module/... \
@@ -209,8 +208,7 @@ jobs:
209208
-//examples/simple_oci_test/... \
210209
-//examples/microservices_architecture/... \
211210
-//test/integration:validate_consumer_deps \
212-
-//test_wit_deps/consumer:check_deps \
213-
--keep_going
211+
-//test_wit_deps/consumer:check_deps
214212
215213
- name: Run Tests
216214
run: bazel test --test_output=errors -- //test/integration:basic_component_build_test //test/integration:basic_component_validation //test/unit:unit_tests

0 commit comments

Comments
 (0)