Skip to content

Render API doc markdown as native godoc in comments#1719

Draft
tanmay-db wants to merge 1 commit into
databricks:mainfrom
tanmay-db:docfmt-native-godoc
Draft

Render API doc markdown as native godoc in comments#1719
tanmay-db wants to merge 1 commit into
databricks:mainfrom
tanmay-db:docfmt-native-godoc

Conversation

@tanmay-db

Copy link
Copy Markdown
Contributor

Regenerated doc comments with genkit's new docfmt renderer, which parses spec descriptions as CommonMark and emits godoc-native comment syntax instead of word-wrapped raw markdown:

  • fenced code examples -> tab-indented godoc code blocks (previously collapsed onto a single line inside literal ``` fences)
  • bullet/numbered lists -> godoc list syntax (previously broken by word-wrap, leaving list markers dangling mid-line)
  • markdown reference links keep their godoc-native [text]: url form

Generated from the exact OpenAPI spec sha the SDK is pinned to (.codegen/_openapi_sha), so the diff is comment-only: every changed line is a // comment line, verified mechanically. The full toolchain (goimports, gofmt, mockery) ran as in the regular pipeline.

Co-authored-by: Isaac

Summary

Why

What changed

Interface changes

Behavioral changes

Internal changes

How is this tested?

Regenerated doc comments with genkit's new docfmt renderer, which
parses spec descriptions as CommonMark and emits godoc-native comment
syntax instead of word-wrapped raw markdown:

- fenced code examples -> tab-indented godoc code blocks (previously
  collapsed onto a single line inside literal ``` fences)
- bullet/numbered lists -> godoc list syntax (previously broken by
  word-wrap, leaving list markers dangling mid-line)
- markdown reference links keep their godoc-native [text]: url form

Generated from the exact OpenAPI spec sha the SDK is pinned to
(.codegen/_openapi_sha), so the diff is comment-only: every changed
line is a // comment line, verified mechanically. The full toolchain
(goimports, gofmt, mockery) ran as in the regular pipeline.

Co-authored-by: Isaac
@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1719
  • Commit SHA: b0adf52733b7c1450420420b95ab8b57adcf2c69

Checks will be approved automatically on success.

@github-actions

Copy link
Copy Markdown

Please ensure that the NEXT_CHANGELOG.md file is updated with any relevant changes.
If this is not necessary for your PR, please include the following in your PR description:
NO_CHANGELOG=true
and rerun the job.

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