docs(sdk): add Go SDK documentation and CI job (3/4) - #11
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 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 |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
ce2b841 to
798a017
Compare
798a017 to
8ec90d2
Compare
8ec90d2 to
b5db057
Compare
b5db057 to
edbf3ed
Compare
Add Fern MDX documentation pages (getting-started, architecture, error-handling, authentication), wire into docs navigation, create go:proto mise task, and add Go job to branch-checks CI workflow. Part 3 of 4 for issue NVIDIA#2044. Assisted-By: 🤖 Claude Code Signed-off-by: Roland Huß <rhuss@redhat.com>
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Applied fixes from bot review comments: - Comments #3563986610-3563986676: fix SPDX copyright year to 2025-2026 - Comment #3563988719: use sb.Status.Phase instead of sb.Status - Comment #3563988730: fix Create API to use (name, spec, labels) signature - Comment #3563988737: fix Exec.Run to use []string command and cast Stdout - Comment #3563988745: fix StaticToken import to v1 package - Comment #3563988756: fix OIDC refresh description accuracy - Comment #3564030817: fix oidc import path in auth docs Assisted-By: 🤖 Claude Code Signed-off-by: Roland Huß <rhuss@redhat.com>
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Lower the minimum Go version to 1.24.0, matching PR #9. Assisted-By: 🤖 Claude Code
Lower the minimum Go version to 1.24.0, matching PR #9. Assisted-By: 🤖 Claude Code
Lower the minimum Go version to 1.24.0, matching PR #9. Assisted-By: 🤖 Claude Code
552e801 to
db75560
Compare
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Applied fixes from bot review comments: - Comment #3564062168: fix TLS field names (InsecureSkipVerify->Insecure, CACertFile->CAFile) - Comment #3564062197: fix error code constant names (NotFound->ErrorNotFound, etc.) - Comment #3564062212: fix retry field name (Backoff->InitialWait) in error-handling - Comment #3564062234: fix retry field name (Backoff->InitialWait) in getting-started - Comment #3564062266: fix error codes table (remove Unauthenticated, add missing codes) - Comment #3564062301: add missing Config() accessor to sub-clients table - Comment #3564256837: update Go version requirement from 1.25 to 1.24 Assisted-By: 🤖 Claude Code
|
This pull request has had no activity for 14 days and is now marked stale. It may be closed in 7 days if there is no further activity. |
Summary
Add Fern MDX documentation and Go CI job for the SDK.
Part 3 of 4 for #2044. Depends on PR 2 (auth/fakes).
What's Included
docs/sdks/go/docs/index.ymlbranch-checks.yml(lint, build, test, proto check)tasks/go.tomlwithgo:proto,go:lint,go:build,go:test7 files