ci: rename relay-interop job to relay-conformance (INTEROP-07) - #77
Merged
Conversation
This job only drives `relay interop --protocol LIN` against RELAY's own embedded LIN golden vectors -- self-consistency against RELAY's own reference conversion, not third-party interoperability against an independent LIN peer. Per SoundMatt/RELAY docs/INTEROP-ARCHITECTURE.md's naming discipline, `*-interop` is reserved for a job that talks to a peer this repo did not build; a step that only invokes RELAY's own CLI is `*-conformance`. Not previously flagged: the design doc's own current-state inventory only called out rust-RCP and rust-LIN as naming violators and missed this one -- re-verified directly against this repo's real ci.yml rather than trusting that inventory, per the doc's own "every claim independently re-checked" standard. Pure rename, no behavior change: confirmed no other job's `needs:` and no branch-protection required-status-check references the old job id (404 -- this repo has no branch protection configured); manual duplicate-job-ID scan (8 job ids, 8 unique) since `yaml.safe_load` alone can't be trusted to catch that class of bug. Verified locally against the pinned relay v2.0.4 build this job installs: `relay conform --strict` and `relay interop --protocol LIN` both still PASS against a real go-lin build. SoundMatt/RELAY THEME-K (#125), docs/INTEROP-ARCHITECTURE.md shipping order item 2. Sign-off: DCO Signed-off-by: Matt Jones <47545907+SoundMatt@users.noreply.github.com>
Owner
Author
|
Merged. Fresh-clone re-verify: Companion fix: SoundMatt/cpp-LIN#53. SoundMatt/RELAY THEME-K (#125), docs/INTEROP-ARCHITECTURE.md shipping order item 2. |
SoundMatt
added a commit
to SoundMatt/cpp-LIN
that referenced
this pull request
Aug 21, 2026
#53) The step only drives `relay interop --protocol LIN` against RELAY's own embedded LIN golden vectors -- self-consistency against RELAY's own reference conversion, not third-party interoperability against an independent LIN peer. Per SoundMatt/RELAY docs/INTEROP-ARCHITECTURE.md's naming discipline, `*-interop` is reserved for a check against a peer this repo did not build; a step invoking only RELAY's own CLI should say so. Not previously flagged: the design doc's own current-state inventory only called out rust-RCP and rust-LIN as naming violators (at the job level) and missed this repo's step-level mislabeling entirely -- re-verified directly against this repo's real ci.yml rather than trusting that inventory. The enclosing job (`relay-conform`) was already correctly named; only this one step's display name was misleading. Pure rename, no behavior change: the underlying `run:` command is unchanged. Manual duplicate-job-ID scan (9 job ids, 9 unique) since `yaml.safe_load` alone can't be trusted to catch that class of bug. Verified locally against the pinned relay v2.0.4 build this job installs: built cpp-lin-cli from a clean cmake+ninja configure, then confirmed `relay conform --strict` and `relay interop --protocol LIN` both still PASS. SoundMatt/RELAY THEME-K (#125), docs/INTEROP-ARCHITECTURE.md shipping order item 2. Companion fix: SoundMatt/go-LIN#77. Sign-off: DCO Signed-off-by: Matt Jones <47545907+SoundMatt@users.noreply.github.com>
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
1 participant
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
The
relay-interopjob only drivesrelay interop --protocol LINagainst RELAY's own embedded LIN golden vectors — self-consistency against RELAY's own reference conversion, not third-party interoperability against an independent LIN peer. Renamed torelay-conformanceperdocs/INTEROP-ARCHITECTURE.md's naming discipline.Not previously flagged: the design doc's own current-state inventory only called out rust-RCP and rust-LIN as naming violators and missed this repo — re-verified directly against this repo's real
ci.yml.Verification
Pure rename, no behavior change:
needs:and no branch-protection required-status-check references the old job id.relay conform --strictandrelay interop --protocol LINboth still PASS against a realgo-linbuild.SoundMatt/RELAY THEME-K (#125),
docs/INTEROP-ARCHITECTURE.mdshipping order item 2.Sign-off: DCO