Skip to content

Add shorter JSON interoperability example commands - #699

Merged
gerardrecinto merged 1 commit into
masterfrom
feat/short-interop-example-commands
Oct 31, 2026
Merged

gerardrecinto merged 1 commit into
masterfrom
feat/short-interop-example-commands

Conversation

@gerardrecinto

Copy link
Copy Markdown
Collaborator

Add shorter entrypoints for the three JSON interoperability examples:

go run ./interop_indexes
go run ./interop_indexes/struct_key_example
go run ./interop_jsondb

The original examples paths and their main.go files still work. Each old and new entrypoint is a thin wrapper around one internal implementation. The example bodies, data locations, output and error behavior are preserved. The secondary-index README now names the real command paths instead of the nonexistent interop_secondary_indexes directory.

Validation:

  • GOWORK=off builds all six original/short package paths and their explicit main.go entrypoints against the published sibling-module versions.
  • The original and short commands run in separate temporary directories. Their output matches after normalizing generated UUIDs only.
  • Both indexed examples read back 500 records and verify category-ascending, price-descending order. The JSON document example verifies the expected person record after its write/read round trip.
  • Targeted go vet, gofmt and git diff --check pass.
  • New Linux and macOS PR checks run the same consumer build and behavior-parity script.

No module versions, workspace configuration or root documentation changed. This PR does not publish a release or change @latest instructions.

Thanks,
Gerard

@gerardrecinto gerardrecinto self-assigned this Oct 29, 2026
@gerardrecinto

Copy link
Copy Markdown
Collaborator Author

/gemini review

Thanks,
Gerard

@github-actions

Copy link
Copy Markdown

Gemini PR Review

Reviewed commit: 421e6bdcd970b7de59b14dc1e14dc70817ad2b1e
Verdict: PASS

@gerardrecinto
gerardrecinto merged commit f6e0eaa into master Oct 31, 2026
32 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant