build: rename module to canonical github.com/ENTERPILOT/GoModel path#457
build: rename module to canonical github.com/ENTERPILOT/GoModel path#457SantiagoDePolonia wants to merge 2 commits into
Conversation
|
Too many files changed for review. ( Bypass the limit by tagging |
|
Important Review skippedToo many files! This PR contains 452 files, which is 302 over the limit of 150. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (452)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
edb13d5 to
3079356
Compare
3079356 to
b29f0f3
Compare
b28fbe3 to
1b13d56
Compare
b29f0f3 to
eebe0a6
Compare
eebe0a6 to
4f670a8
Compare
Makes GoModel fetchable as a Go module (go get, versioned requires, sumdb integrity) so external parties can build on the extension API without a replace directive. Pure mechanical rename: go.mod, all import paths, and version ldflags in Makefile/Dockerfile/goreleaser. No behavior change. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Documents the ext rewriter contract (fail-closed, ordering, concurrency, credential-header redaction, audit keeps the original body), run.Run embedding, cross-module ldflags versioning, and Prometheus/config guidance. The ext/run API is marked experimental. Lands with the canonical module path because the embedding recipe depends on go-gettable imports. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
4f670a8 to
80eff53
Compare
What this does
Rename + embedding docs. Split from the original combined PR at maintainer request — the extension API and embeddable bootstrap live in #479, which this PR is stacked on (merge #479 first; this diff shows two commits on top).
build:renames the module fromgomodeltogithub.com/ENTERPILOT/GoModel: go.mod, every import path, and version ldflags (Makefile, Dockerfile, goreleaser). No behavior change — pure find/replace.docs:addsdocs/extensions.md, the extension/embedding guide. It lands here rather than in feat: public extension API and embeddable bootstrap #479 because the embedding recipe (imports,require, cross-module ldflags) only becomes practical once the module is fetchable. Theext/runAPI is explicitly marked experimental.Why
A module named
gomodelcannot be fetched bygo get(no host in the path), so nobody can depend on GoModel as a library: external builds on the #479 extension API would forever need a local checkout plus areplacedirective, unversioned and without sumdb integrity. This rename is the enabling step for versioned, checksummed, transitive consumption —require github.com/ENTERPILOT/GoModel vX.Y.Z.Reversibility & timing
Merging this commits us to almost nothing; the ratchet is the first canonical tag (module-proxy versions are cached immutably and adoption makes the path permanent). Recommended: merge when the PR queue is quiet — this conflicts trivially with every open PR touching imports — and treat tagging/announcing as the real decision gate. A
retractdirective remains available if we change course before adoption.Testing
The stacked tree is byte-identical to the previously fully-verified combined branch: full unit suite, e2e, dashboard tests, and golangci-lint green; downstream consumer build verified.
🤖 Generated with Claude Code