- Notifications
You must be signed in to change notification settings - Fork32
feat: Package scaffolding & setup#454
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
d33d829 to14ee002Compare| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| __version__="0.0.1" |
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.
nit: should we start at 0.1.0? or will release please auto-update this?
| @@ -0,0 +1,27 @@ | |||
| #Toolbox ADK Integration | |||
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.
Can we fill this out like the LangChain version?
| [project] | ||
| name = "toolbox-adk" | ||
| dynamic = ["version"] | ||
| description = "ADK Integration for MCP Toolbox" |
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.
nit: do we want to use "Agent Development Kit"?
14ee002 to7da5271Compare
Uh oh!
There was an error while loading.Please reload this page.
Initial scaffolding for
toolbox-adkpackage.Note
We've used
TOOLBOX_IDENTITYin the initialREADMEfor now. This will be changed in the subsequent PR:#456.Note
Integration tests are expected to fail in this branches as expected and that they would pass ultimately in the subsequent PR:#458