From 60511bd4546c917e305ecd60f0c4a653e20903eb Mon Sep 17 00:00:00 2001 From: Matt <47545907+SoundMatt@users.noreply.github.com> Date: Thu, 30 Jul 2026 15:31:52 -0700 Subject: [PATCH] fix: bump CI relay pin to v2.0.4 under the new /v2 module path MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The conformance-gate job installed the reference relay CLI via `go install github.com/SoundMatt/RELAY/cmd/relay@v1.11.0`, which was already four minor versions stale (v1.12–v1.14 shipped since) and, as of RELAY's v2.0 MAJOR bump, is no longer installable at all: v2.0 changed the module path to `github.com/SoundMatt/RELAY/v2` per Go's semantic import versioning rules (SoundMatt/RELAY#70), so the old import path can't resolve v2.x releases at all. Bump the pin to `github.com/SoundMatt/RELAY/v2/cmd/relay@v2.0.4`. Reviewed the intervening spec changes (CHANGELOG v1.12–v2.0) for any newly-applicable conformance requirements: - v1.12: added "c" as a valid CLI language value — not applicable, rust-LIN already reports "rust". - v1.13: fixed `relay interop`/`relay conform --strict` CLI bugs, and documented HealthProvider/MetricsProvider/Drainer for the C++/Rust bindings — these are optional, capability-declared interfaces (spec §9); rust-LIN already implements HealthProvider/MetricsProvider on VirtualBus. - v1.14: expanded the §13.7.2 module-name registry with RCP-specific module names (avtp, acf, lifecycle, regmap, discovery, request, fragment) and standardized DDS RTPS internal naming — neither applies to a LIN implementation. - v2.0: replaced RCP's canonical types with the real TC18 protocol — RCP-specific, does not touch LIN. Built the reference `relay` v2.0.4 CLI locally the same way CI now will and ran it against a release build of rust-lin: - `relay conform --strict ./target/release/rust-lin` → PASS - `relay interop --strict --protocol LIN ./target/release/rust-lin` → PASS (3/3 golden vectors EQUIVALENT) No new conformance gap, so no source changes and no crate version bump — consistent with the precedent set by the rust-FuSa CI-pin-only fix in #11, which also didn't bump the crate version. Full local verification also run and green: cargo build --release, cargo test (46 unit/integration + 2 doctests), cargo clippy -D warnings, cargo fmt --check, and the ungated rust-FuSa v0.3.10 gates (rsfusa qualify: 16/16, rsfusa release). Signed-off-by: Matt <47545907+SoundMatt@users.noreply.github.com> --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 9a49689..906df3f 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -251,7 +251,7 @@ jobs: run: cargo build --release --bin rust-lin --locked - name: Install relay - run: go install github.com/SoundMatt/RELAY/cmd/relay@v1.11.0 + run: go install github.com/SoundMatt/RELAY/v2/cmd/relay@v2.0.4 - name: RELAY conformance (strict) run: relay conform --strict ./target/release/rust-lin