feat(go): initialize a minimal Go module and CI workflow for the upcoming Go SDK#829
feat(go): initialize a minimal Go module and CI workflow for the upcoming Go SDK#829ZekiLiu wants to merge 1 commit intoapache:mainfrom
Conversation
794236d to
82ae651
Compare
|
The previous license header check was failing because the header text didn’t match the expected template exactly (it was missing a space). I’ve updated the header formatting to match the template so the licencse check can pass. |
4fed154 to
5236a75
Compare
|
Map go.* and go.mod to use the double-slash (//) comment style for proper header injection. Reference: apache/fory/licenserc.toml And verify these changes locally using Docker with the following command docker run --rm \
-v "$PWD:/github/workspace" \
-w /github/workspace \
-e "GITHUB_WORKSPACE=/github/workspace" \
-e "CI=true" \
ghcr.io/korandoru/hawkeye-native:v4.0.5 \
check --config licenserc.tomlResult: |
5236a75 to
81349b7
Compare
|
Hi @ZekiLiu, thank you for doing so much. When the |
…ming Go SDK * feat: initialize go module with v1.21 * ci: implement automated test workflow * fix: license header format and configuratoin * chore: move go/* to go/graphar
81349b7 to
93e9d57
Compare
Got it, thank you! |
Introduce a minimal Go module and CI workflow (Go 1.21) for the upcoming Go SDK.
Reason for this PR
This PR bootstraps the initial Go SDK skeleton under
go//graphar/and sets up CI to run Go tests.It is intended to provide a minimal, reviewable baseline so the community can iterate on the Go SDK design and implementation incrementally. #828
What changes are included in this PR?
go/graphar/(go/graphar/go.mod, Go 1.21).graphar.Clientandgraphar.NewClient())..github/workflows/go.yml) to rungo teston changes ingo/graphar/**.go/graphar/Makefilewithmake testfor a consistent developer workflow (aligned with other language subprojects).Are these changes tested?
Yes.
go test ./...(executed in CI via.github/workflows/go.yml).Are there any user-facing changes?
No. This PR only introduces the initial project skeleton and CI for the Go SDK; it does not implement GraphAr format reading/writing yet.