Skip to content

feat: add standalone "Test examples" CI step and complete documentation coverage - #3

Merged
Pijukatel merged 1 commit into
masterfrom
claude/eager-wright-5pyas6
Jun 22, 2026
Merged

Pijukatel merged 1 commit into
masterfrom
claude/eager-wright-5pyas6

Conversation

@Pijukatel

Copy link
Copy Markdown
Collaborator

Summary

Brings the Go client into compliance with the updated client requirements (documentation/examples). This is a CI/docs compliance change only — the client is already at OpenAPI spec v2-2026-06-18T095846Z, so there is no spec bump and no public-interface change.

Changes

  • Standalone Test examples CI step.github/workflows/go-integration-tests.yml now has a step named exactly Test examples that actually runs the documentation example code: each of the 7 example programs is executed end-to-end via go run ./examples/<name> against the live API, guarded by the APIFY_TOKEN secret. The Integration tests step now skips the example/doc-snippet tests (-skip '^(TestExample|TestDocSnippets)') so they run only in the dedicated step. Triggers extended to docs/** and README.md.
  • All in-documentation snippets are compiled and format-checked — added tests/docs_snippets_test.go, which extracts every fenced Go block from README.md and docs/*.md, asserts each is gofmt-formatted, and compiles each against the module. Fixed the real formatting/compile issues it surfaced and turned the custom-HTTP-transport README snippet into a complete, compilable program.
  • Documentation completeness — documented field/type/meaning for every public option struct and response model read by the README quick start and the examples, and stated the closed enums definitively (run/build status ActorJobStatus — corrected an incomplete list, WebhookEventType, VersionSourceType, SourceCodeFileFormat, pricing/response-format/permission/ownership), each verified value-by-value against the OpenAPI spec.
  • Bumped library version 0.2.00.2.1 and recorded the change in CHANGELOG.md.

Verification

  • gofmt -l ., go vet ./..., go build ./... clean.
  • Unit tests pass; 29 integration tests pass against the live API.
  • Test examples step passes: the doc-snippet checks plus all 7 example programs run end-to-end.

Generated by Claude Code

Align the client with the updated client requirements:

- CI: add a standalone `Test examples` step that actually runs the
  documentation example code end-to-end (each of the 7 example programs is
  executed via `go run ./examples/<name>` against the live API, guarded by
  the APIFY_TOKEN secret). The `Integration tests` step now skips the
  example/doc-snippet tests so they run only in the dedicated step.
- Docs: add tests/docs_snippets_test.go, which extracts every fenced Go
  block from README.md and docs/*.md, asserts gofmt formatting, and compiles
  each snippet against the module. Fixed the formatting/compile issues it
  surfaced and made the custom-transport snippet a complete program.
- Docs: document field/type/meaning for every public option struct and
  response model used by the README quick start and examples, and state the
  closed enums definitively (run/build status, webhook event types, version
  source types, etc.), verified against the OpenAPI spec.
- Bump version to 0.2.1 and record the change in CHANGELOG.md.
@Pijukatel Pijukatel added the adhoc Ad-hoc unplanned task added during the sprint. label Jun 22, 2026
@Pijukatel Pijukatel added this to the v0 milestone Jun 22, 2026
@github-actions github-actions Bot added t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics. labels Jun 22, 2026
@Pijukatel
Pijukatel merged commit f095192 into master Jun 22, 2026
2 of 3 checks passed
@Pijukatel
Pijukatel deleted the claude/eager-wright-5pyas6 branch July 8, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants