docs(go): named dep groups for link deps/runtime_deps#57
Merged
Conversation
Reflects hephbuild/heph#140 (commit 5196c8246f23d0ae81121edf0bdb4efe831bb9d7), which widens the go provider's `link` deps/runtime_deps provider_state fields from list[string] to string | list[string] | {group: ...}. Updates both the docs page and its heph-go skill reference twin per the reference-drift rule.
⚡ Cloudflare Pages preview
|
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.
Summary
5196c82), which widens the go provider'slinkdeps/runtime_depsprovider_statefields fromlist[string]tostring | list[string] | {group: string | list[string]}, letting BUILD authors name dep groups instead of always landing in one bucket.website/docs/plugins/go.md(Link configuration section) with the new type signature and a "Naming dep groups" subsection.plugins/heph-go/skills/heph-go/references/go-plugin.md(theheph-goClaude Code skill's bundled reference) to match, per the reference-drift rule in this repo'sCLAUDE.md, plus a troubleshooting row for group-name collisions.Note: the upstream PR description says named groups are emitted with a
link_deps_/link_runtime_deps_prefix, but the actual merged code (link_group_keyincrates/plugin-go/src/plugingo/target_bin.rs) and its own tests (test_link_deps_named_groups_are_verbatim) show named groups are staged verbatim, not prefixed — only the default (unnamed) group uses thelink_deps/link_runtime_depskey. This PR documents the actual, tested behavior in the merged code rather than the PR body's description.Test plan
hephbuild/heph(crates/plugin-go/src/plugingo/provider.rs,crates/plugin-go/src/plugingo/target_bin.rs).Generated by Claude Code