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.

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

fix: pin autogen and langgraph-prebuilt #1376

Closed
wants to merge 1 commit into from

Conversation

angrybayblade
Copy link
Collaborator

@angrybayblade angrybayblade commented Feb 27, 2025

Important

Pin langgraph-prebuilt to >=0.1.0 and add ag2[openai] in examples.yml.

  • Dependencies:
    • Pin langgraph-prebuilt to version >=0.1.0 in examples.yml.
    • Add ag2[openai] to the list of installed packages in examples.yml.

This description was created by Ellipsis for 000e2b2. It will automatically update as commits are pushed.

Copy link

vercel bot commented Feb 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
composio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 27, 2025 10:42am

@@ -94,7 +94,7 @@ jobs:
python -m uv pip install plugins/${{ matrix.package }}
python -m uv pip install '.[all]'
python -m uv pip install 'numpy<2' python-dotenv
python -m uv pip install unstructured
python -m uv pip unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'

Choose a reason for hiding this comment

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

Invalid pip command syntax - python -m uv pip unstructured is missing install command which will cause installation failure

📝 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
python -m uv pip unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'
python -m uv pip install unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'

@angrybayblade angrybayblade deleted the fix/example-test-deps branch February 27, 2025 10:42
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Reviewed everything up to 000e2b2 in 1 minute and 43 seconds

More details
  • Looked at 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 drafted comments based on config settings.
1. .github/workflows/examples.yml:97
  • Draft comment:
    It appears the 'install' keyword was omitted in the pip command. Should this be 'python -m uv pip install ag2[openai] "langgraph-prebuilt>=0.1.0"'?
  • Reason this comment was not posted:
    Marked as duplicate.
2. .github/workflows/examples.yml:97
  • Draft comment:
    Typo: The pip command is missing the 'install' keyword. It should be 'python -m uv pip install unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'' to maintain consistent syntax with the other pip commands.
  • Reason this comment was not posted:
    Marked as duplicate.

Workflow ID: wflow_PCIssFec5tErE9Xn


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@@ -94,7 +94,7 @@ jobs:
python -m uv pip install plugins/${{ matrix.package }}
python -m uv pip install '.[all]'
python -m uv pip install 'numpy<2' python-dotenv
python -m uv pip install unstructured
python -m uv pip unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing 'install' in the pip command. Ensure it uses 'pip install' to correctly install packages.

Suggested change
python -m uv pip unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'
python -m uv pip install unstructured ag2[openai] 'langgraph-prebuilt>=0.1.0'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant