Skip to content
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.

Sign up for GitHub

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

Update autogen.mdx #1184

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
+2 −10
Open

Update autogen.mdx #1184

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b0c639c
Update autogen.mdx
abhishekpatil42 Jan 13, 2025
a8b1a1d
Update openai.spec.ts
abhishekpatil42 Jan 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/framework/autogen.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
title: "Using Composio With Autogen"
title: "Using Composio With Autogen test"
Copy link

Choose a reason for hiding this comment

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

The title has been changed from "Using Composio With Autogen" to "Using Composio With Autogen test". If this was unintentional, it should be reverted. If intentional, consider if "test" is necessary or if a more descriptive title could be used.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
title: "Using Composio With Autogen test"
title: "Using Composio With Autogen"

Copy link
Contributor

Choose a reason for hiding this comment

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

The title change to "Using Composio With Autogen test" seems unintentional. Consider reverting it to "Using Composio With Autogen" for consistency and professionalism.

Copy link
Collaborator

Choose a reason for hiding this comment

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

The addition of 'test' to the documentation title appears to be unintentional and should be reverted. This is a production documentation file that is referenced across multiple other docs files, and should maintain a professional title.

Copy link

Choose a reason for hiding this comment

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

The title change from "Using Composio With Autogen" to "Using Composio With Autogen test" in the autogen.mdx file seems like it might be unintentional and should be reverted or confirmed as intended.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
title: "Using Composio With Autogen test"
title: "Using Composio With Autogen"

sidebarTitle: "Autogen"
description: "Integrate Composio with Autogen agents to let them seamlessly interact with external Apps"
---
Expand Down Expand Up @@ -74,4 +74,4 @@ response = user_proxy.initiate_chat(chatbot, message=task)
print(response.chat_history)
```
</Step>
</Steps>
</Steps>
8 changes: 0 additions & 8 deletions js/src/frameworks/openai.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,6 @@ describe("Apps class tests", () => {
});
});

it("get tools", async () => {
const tools = await openAIToolset.getTools({
apps: ["github"],
});

expect(tools).toBeInstanceOf(Array);
});

it("check if tools are coming", async () => {
const tools = await openAIToolset.getTools({
actions: ["GITHUB_GITHUB_API_ROOT"],
Expand Down