-
Notifications
You must be signed in to change notification settings - Fork 3
feat(NGUI-223): A2A protocol support (dev preview) - package and container image #149
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
|
The code looks good. Just a general comment that I would make README.md less code heavy and more text. I don't think it makes a lot of sense having the same code in readme and in separate python files. They will surely get out of sync at some point and it would be easier to just point to the right file and explain from a high level what it is doing or what it is useful for. |
5de3775 to
3c1a286
Compare
velias
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.
Looks good, few notes, mainly towards documentation or additional features
A2A protocol support.
See README.md
Related: lightspeed-core/lightspeed-stack#727