Skip to content

Document Agent Skills feature in ADK Go#1687

Merged
koverholt merged 3 commits into
google:mainfrom
foxfrikses:update-skills-doc-for-adk-go
Apr 28, 2026
Merged

Document Agent Skills feature in ADK Go#1687
koverholt merged 3 commits into
google:mainfrom
foxfrikses:update-skills-doc-for-adk-go

Conversation

@foxfrikses

Copy link
Copy Markdown
Contributor

No description provided.

@netlify

netlify Bot commented Apr 27, 2026

Copy link
Copy Markdown

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit 548a3ad
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/69f0e83f23c0ce0008270487
😎 Deploy Preview https://deploy-preview-1687--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@koverholt koverholt self-requested a review April 27, 2026 16:20

@koverholt koverholt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the PR to add Go support to the Skills docs! The API usage and import paths look correct against the adk-go v1.2.0 source.

Left some comments, mostly around small code issues in the Go snippets and a few structural/formatting suggestions. The main substantive feedback is on the "Provision Skills dynamically" section, which could use some reworking.

Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
Comment thread docs/skills/index.md Outdated
@foxfrikses foxfrikses force-pushed the update-skills-doc-for-adk-go branch from 4f30def to 4720cc1 Compare April 28, 2026 06:39
@foxfrikses foxfrikses force-pushed the update-skills-doc-for-adk-go branch from 4720cc1 to c073c48 Compare April 28, 2026 06:43
@foxfrikses

Copy link
Copy Markdown
Contributor Author

Thank you for the review!

I believe I addressed all your comments.

@foxfrikses foxfrikses requested a review from koverholt April 28, 2026 06:52
Comment thread docs/skills/index.md Outdated

@koverholt koverholt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for working through all of the issues. The updated content LGTM, merging!

@koverholt koverholt merged commit 5875db6 into google:main Apr 28, 2026
6 checks passed
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.

3 participants