-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(api)!: deprecate toolgroup and tool_runtime apis
#4249
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
toolgroup and tool_runtime apis
toolgroup and tool_runtime apistoolgroup and tool_runtime apis
da0fb7b to
0250201
Compare
We wait for one "minor" release right now -- so we will mark these for deprecation for 0.4.x and then remove them for 0.5.x |
|
I wonder why the Stainless SDK build checks did not run on this PR. Something about CI might be wonky given the llama-stack-api move? Maybe the wildcards for actions need to be updated @cdoern ? |
Closes llamastack#4233 and llamastack#4061 (partially) Signed-off-by: Jaideep Rao <[email protected]>
0250201 to
b54df67
Compare
✱ Stainless preview buildsThis PR will update the
|
|
not sure why the API conformance test is failing all of a sudden |
|
I think stainless wasn't running + CI was green due to the lack of GH actions over the last week @jaideepr97 @ashwinb , this now looks accurate to what I'd expect from a PR like this. |
toolgroup and tool_runtime apistoolgroup and tool_runtime apis
|
conformance should pass because of the |
|
lgtm |
What does this PR do?
marks
toolgroupandtool_runtimeAPIs for deprecationCloses #4233 and #4061 (partially)
How long do we wait before we remove deprecated APIs?
Test Plan