Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@marklysze
Copy link
Collaborator

Why are these changes needed?

Claude Code is reviewing draft PRs which can create too much activity, disabling for drafts.

Related issue number

N/A

Checks

@joggrbot
Copy link

joggrbot bot commented Oct 15, 2025

📝 Documentation Analysis

All docs are up to date! 🎉


✅ Latest commit analyzed: f6f28ec | Powered by Joggr

@marklysze marklysze marked this pull request as draft October 15, 2025 19:16
@marklysze marklysze marked this pull request as ready for review October 15, 2025 19:29
@marklysze marklysze merged commit 13559a5 into main Oct 15, 2025
18 of 19 checks passed
@marklysze marklysze deleted the chore/disable-claude-review-drafts branch October 15, 2025 19:31
@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 41 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-merge-queue bot pushed a commit that referenced this pull request Oct 16, 2025
… chat and group chat APIs (#2145)

* Enable list[dict] type for message['content'] for two-agent chat and group chat APIs

* Update test marker

* Add back test_custom_client.py

* skip test_multimodal_integration.py  from no llm test

* Update skip llm test

* update

* Update e2e test

* enable manual picked branch for workflow

* Fixed Tool Executor UnboundLocalError (autogen/agentchat/group/group_tool_executor.py:149-157)

  - Issue: tool_message variable was not initialized before the loop, causing UnboundLocalError when no tool calls were present
  - Fix: Added tool_message = None initialization and improved condition checking with message.get('tool_calls') instead of 'tool_calls' in message

* Verified Core Import Isolation

  - Marker-based filtering system in test/conftest.py is working correctly
  - No Google dependency import errors in OpenAI tests
  - Proper test isolation achieved between different LLM providers

* Add responses client to testing config

* Fix test

* Disable Claude code review on Draft PRs (#2147)

---------

Co-authored-by: Mark Sze <[email protected]>
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