Repository navigation
ci(go): validate WASM routing bindings - #1131
Draft
afourniernv wants to merge 2 commits into
Draft
afourniernv wants to merge 2 commits into
afourniernv wants to merge 2 commits into
Conversation
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
from
October 23, 2026 17:14
f5047f3 to
3c2f009
Compare
afourniernv
force-pushed
the
codex/go-review-04b2-wasm-routing
branch
from
October 23, 2026 17:14
61db4a5 to
beb73e4
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
from
October 26, 2026 16:36
3c2f009 to
8efff51
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
from
October 26, 2026 17:07
8efff51 to
b8ddcf9
Compare
afourniernv
changed the base branch from
codex/go-review-04b2-wasm-routing
to
codex/go-backend-reporting
October 26, 2026 17:08
afourniernv
force-pushed
the
codex/go-backend-reporting
branch
from
October 26, 2026 19:05
56939c4 to
c0874d1
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
2 times, most recently
from
October 26, 2026 20:12
52441fa to
29fa359
Compare
afourniernv
force-pushed
the
codex/go-backend-reporting
branch
from
October 26, 2026 20:12
c0874d1 to
25393da
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
from
October 27, 2026 20:26
4cc16f3 to
62eaeaf
Compare
afourniernv
force-pushed
the
codex/go-backend-reporting
branch
from
October 27, 2026 23:44
00b7f64 to
cd21af7
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
from
October 27, 2026 23:44
62eaeaf to
7088a95
Compare
afourniernv
force-pushed
the
codex/go-backend-reporting
branch
from
October 28, 2026 00:09
cd21af7 to
8d5347f
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
2 times, most recently
from
October 28, 2026 00:40
20b5a1b to
47fdd43
Compare
afourniernv
force-pushed
the
codex/go-backend-reporting
branch
2 times, most recently
from
October 29, 2026 19:25
2000106 to
39a777b
Compare
Signed-off-by: Alex Fournier <afournier@nvidia.com>
Signed-off-by: Alex Fournier <afournier@nvidia.com>
afourniernv
force-pushed
the
codex/go-backend-reporting
branch
from
October 30, 2026 16:53
39a777b to
420c27c
Compare
afourniernv
force-pushed
the
codex/go-review-06b-ci-wasm
branch
from
October 30, 2026 16:53
47fdd43 to
9157514
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Add a separate WASM job to the Go validation workflow. Build the guest, run tests and vet with
CGO_ENABLED=0, then run the WASM tests with Go race detection.Why
The default WASM backend needs explicit coverage without cgo. Separate native and WASM jobs make failures easier to identify.
How tested
Checked the refreshed stack locally with Go 1.26.2 on macOS arm64:
GOEXPERIMENT=cgocheck2and a forced rebuild.Native links emitted local macOS deployment-target warnings; tests passed. No live provider calls or publishing. GitHub checks run separately.
Notes for reviewers
Stacked on #1147, which adds Go backend reporting. Still a draft.
The guest build commands are inlined in this workflow and documented in the README. Validation only: no artifact uploads, releases, or publishing. The Go version comes from
go.mod.Adapted from Nick DiZazzo's dual-engine contribution. His original PR and branch are unchanged.