Skip to content

Commit f0db61e

Browse files
committed
fix: properly exclude WIT-enabled Go targets from CI builds
Replace wildcard //examples/go_component/... with explicit target list to avoid building WIT-enabled targets that fail due to upstream TinyGo limitations. Explicit inclusions: - calculator_component_debug (WASI CLI, no WIT) - calculator_manual (no WIT specified) - http_service_component (WASI CLI, no WIT) - simple_test, simple_wasi, multi_file_test (basic Go components) - WIT library targets (calculator_wit, etc.) Excluded WIT-enabled targets: - calculator_component, calculator_simple - calculator_with_bindings, calculator_simple_binding This ensures clean CI builds while maintaining all working functionality.
1 parent 1f01603 commit f0db61e

File tree

1 file changed

+22
-10
lines changed

1 file changed

+22
-10
lines changed

.github/workflows/ci.yml

Lines changed: 22 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,17 @@ jobs:
103103
run: |
104104
# Build core working targets including JavaScript components
105105
bazel build --keep_going -- \
106-
//examples/go_component/... \
106+
//examples/go_component:calculator_component_debug \
107+
//examples/go_component:calculator_docs \
108+
//examples/go_component:calculator_manual \
109+
//examples/go_component:http_service_component \
110+
//examples/go_component:http_service_docs \
111+
//examples/go_component:multi_file_test \
112+
//examples/go_component:simple_test \
113+
//examples/go_component:simple_wasi \
114+
//examples/go_component:calculator_wit \
115+
//examples/go_component:http_service_wit \
116+
//examples/go_component:simple_calculator_wit \
107117
//examples/basic/... \
108118
//examples/simple_module/... \
109119
//examples/cli_tool_example/... \
@@ -127,10 +137,6 @@ jobs:
127137
-//examples/cpp_component/multi_component_system:analytics_service \
128138
-//tools/checksum_updater_wasm/... \
129139
-//tools/ssh_keygen:ssh_keygen_test \
130-
-//examples/go_component:calculator_component \
131-
-//examples/go_component:calculator_simple \
132-
-//examples/go_component:calculator_with_bindings \
133-
-//examples/go_component:calculator_simple_binding \
134140
135141
- name: Run Tests
136142
run: bazel test --test_output=errors -- //test/integration:basic_component_build_test //test/integration:basic_component_validation //test/unit:unit_tests //test/wkg/unit:smoke //test/js:test_hello_js_component_provides_info //test/js:test_calc_js_component_provides_info //test/js:test_npm_dependencies_installation
@@ -196,7 +202,17 @@ jobs:
196202
# Build core working targets using explicit inclusions
197203
# Same approach as Linux but with macOS-specific exclusions
198204
bazel build --keep_going -- \
199-
//examples/go_component/... \
205+
//examples/go_component:calculator_component_debug \
206+
//examples/go_component:calculator_docs \
207+
//examples/go_component:calculator_manual \
208+
//examples/go_component:http_service_component \
209+
//examples/go_component:http_service_docs \
210+
//examples/go_component:multi_file_test \
211+
//examples/go_component:simple_test \
212+
//examples/go_component:simple_wasi \
213+
//examples/go_component:calculator_wit \
214+
//examples/go_component:http_service_wit \
215+
//examples/go_component:simple_calculator_wit \
200216
//examples/basic/... \
201217
//examples/simple_module/... \
202218
//examples/cli_tool_example/... \
@@ -226,10 +242,6 @@ jobs:
226242
-//tools/checksum_updater_wasm/... \
227243
-//test/integration:wasi_component_wasm_lib_release_host \
228244
-//test/integration:service_b_component_wasm_lib_release_host \
229-
-//examples/go_component:calculator_component \
230-
-//examples/go_component:calculator_simple \
231-
-//examples/go_component:calculator_with_bindings \
232-
-//examples/go_component:calculator_simple_binding \
233245
234246
- name: Run Tests
235247
run: bazel test --test_output=errors -- //test/integration:basic_component_build_test //test/integration:basic_component_validation //test/unit:unit_tests //test/js:test_hello_js_component_provides_info //test/js:test_calc_js_component_provides_info //test/js:test_npm_dependencies_installation

0 commit comments

Comments
 (0)