docs: refresh guide and prepare wicked-sqlc v2.4.0 - #19
Merged
Conversation
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
Refresh GUIDE for the merged wicked-sqlc architecture and prepare the v2.4.0
source-release notes. This PR does not create a release tag or publish a release.
Architecture and Key Decisions
optional CGO, and the existing query-comment/backend contract.
list. Link to bookstore's tested runtime dependencies without upgrading them.
docs/releases/v2.4.0.mdwith upgrade instructions, compatibilityevidence, and known limits. Keep the existing source-release distribution
model; no new workflows or binary attachments.
Risks and Remaining Work
Merge this PR before tagging. Then verify the exact merged main SHA and checks,
ensure v2.4.0 remains unused, create the tag and GitHub Release, and verify a fresh
tag-clone installation. The release is not published by this PR.
The earlier Trex full-local-build memory limitation remains documented; no full
Alva service-stack E2E or downstream rollout is claimed.
Breaking Changes
None in this documentation PR.
Database Migrations
None.
Verification
git diff --checkpassed; only GUIDE, release notes, and the changelog change.GOMAXPROCS=2 GOBIN=/tmp/sqlc-v2.4-release.jAOb4e/cgo make install COMMIT_HASH=v2.4.0GOMAXPROCS=2 GOBIN=/tmp/sqlc-v2.4-release.jAOb4e/nocgo make install CGO_ENABLED=0 COMMIT_HASH=v2.4.0v2.4.0-wicked-fork.GOMAXPROCS=2 go test -p 2 -count=1 ./scripts ./internal/cmd -run 'Test(ReleaseVersionMatchesGeneratedCode|Wicked)'passed.node /tmp/sqlc-v2.4-release.jAOb4e/verify.mjspassed both modes' bookstoregenerate/diff, 17-file parity except version comments, CLI/header versions,
documentation links/fences, and unchanged README checks.
444685d1,GOMAXPROCS=2 go test -p 2 -count=1 ./pkg/repos/...compiled all five generatedpackages (no test files).
make lint-fixtarget exists. Full local runtime/E2E was not rerun forMarkdown-only changes; the existing PR CI includes the runtime checks.
Merge Order
This preparation PR first, then source-release publication from verified main.
Related PRs
Follows merged sqlc #17, #18 and Stumble/bookstore#2.