-
Notifications
You must be signed in to change notification settings - Fork 668
Update adk-go docs with launcher changes #967
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Thanks so much for updating. I ran a build script and it gives the following issues: |
ivanmkc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left comments about build errors
| } | ||
|
|
||
| agent, err := llmagent.New(llmagent.Config{ | ||
| timeAgent, err := llmagent.New(llmagent.Config{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be agent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it cannot be agent, because package agent is used in lines 83.
Apparently my change was removed in this 9e30396 merge commit, but I restored it.
|
Regarding the build: seems like your script used older version @ivanmkc could you share your script so I can double check its behavior? |
Hi, I added this PR with the scripts to test, which should help: #973 |
|
DO NOT MERGE: This docs update indicates code changes for ADK Go launcher. HOWEVER, there is nothing to indicate an official release happened:
Pinging the Go team internally for an update. |
|
@joefernandez @ivanmkc adk-go 0.2.0 is released https://pkg.go.dev/google.golang.org/adk@v0.2.0 |
|
All snippets should be buildable except for one ( We actually removed Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this quickstart is now INCOMPATIBLE with release v0.1.0, add this requirement:
* Go 1.24.4 or later
* ADK Go v0.2.0 or later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added.
ivanmkc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it builds then looks good to me.
|
For reference (built with adk-go 0.2.0) @ivanmkc I'd only would like to ask for help with We removed |
46ee9e1 to
ff0c065
Compare
|
Mere nems ka kitana account he kese kare |

It addresses 2 issues:
foo(args) res->foo(args) (res, error)