Skip to content

Conversation

@Joibel
Copy link
Member

@Joibel Joibel commented Nov 24, 2025

Motivation

People use the GO SDK but it's very sparsely documented. Lets make the on-ramp slightly easier for this with a full documentation page about it and some examples

Modifications

Added some (tested) examples which also (except for the difficult to run auth tests) run as tests also to prove they're not very broken.

Introduced snipdoc so that the docs page actually includes tested code from the examples.

Moved kubernetes client config from non-standard .kubeconfig to .kube/config as this is where the examples expect it.

Verification

See above, new test suite.

Documentation

This is principally new documentation.

Re-did the client-libraries page to make it a bit more verbose in anticipation of this being a more worked on area.

@Joibel Joibel added area/sdks area/docs Incorrect, missing, or mistakes in docs labels Nov 24, 2025
@Joibel Joibel force-pushed the gosdk branch 2 times, most recently from 334dda0 to 605ff9f Compare November 24, 2025 13:25
@Joibel Joibel marked this pull request as draft November 24, 2025 13:26
@Joibel Joibel requested a review from Copilot November 24, 2025 13:26
Copilot finished reviewing on behalf of Joibel November 24, 2025 13:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds comprehensive documentation and examples for the Argo Workflows Go SDK, making it easier for developers to get started with the SDK.

Key Changes

  • Added 5 working Go SDK examples with tests (basic-workflow, watch-workflow, workflow-template, grpc-client, alternate_auth)
  • Integrated snipdoc for including tested code snippets in documentation
  • Added comprehensive README files for each example covering setup, usage, and best practices
  • Updated CI/CD to test the Go SDK examples

Reviewed changes

Copilot reviewed 26 out of 31 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
snipdoc-config.yml Configuration for snipdoc tool to extract code snippets from examples
sdks/go/*/go.mod Go module files for each example (contains invalid Go version)
sdks/go/*/main.go Example implementations for various SDK usage patterns
sdks/go/*/README.md Documentation for each example with usage instructions
sdks/CONTRIBUTING.md Minor wording update for SDK contributions
mkdocs.yml Added go-sdk-guide.md to documentation navigation
.spelling Added gRPC and kubeconfig to spelling allowlist
.github/workflows/ci-build.yaml Added test-go-sdk job to CI pipeline

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Joibel Joibel force-pushed the gosdk branch 3 times, most recently from 63332a9 to f735a6a Compare November 24, 2025 16:23
@Joibel Joibel marked this pull request as ready for review November 25, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Incorrect, missing, or mistakes in docs area/sdks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant