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

Skip to content

Conversation

@davorrunje
Copy link
Collaborator

Why are these changes needed?

Related issue number

Closes #1265

Checks

@davorrunje davorrunje requested a review from rjambrecic March 7, 2025 05:48
@davorrunje davorrunje self-assigned this Mar 7, 2025
@marklysze marklysze added this pull request to the merge queue Mar 7, 2025
Merged via the queue into main with commit c9cf3a3 Mar 7, 2025
17 checks passed
@marklysze marklysze deleted the fix-issubclass-checking-error branch March 7, 2025 20:55
@codecov
Copy link

codecov bot commented Mar 7, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
autogen/tools/dependency_injection.py 50.00% 2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (23769ff) and HEAD (c00dad3). Click for more details.

HEAD has 1268 uploads less than BASE
Flag BASE (23769ff) HEAD (c00dad3)
core-without-llm 14 1
3.9 83 0
ubuntu-latest 146 1
3.13 85 0
optional-deps 141 0
commsagent-slack 9 0
3.10 96 0
commsagent-telegram 9 0
3.11 64 1
macos-latest 104 0
3.12 36 0
windows-latest 114 0
commsagent-discord 9 0
retrievechat 15 0
jupyter-executor 9 0
retrievechat-mongodb 10 0
graph-rag-falkor-db 6 0
browser-use 7 0
twilio 9 0
retrievechat-qdrant 14 0
crawl4ai 13 0
retrievechat-pgvector 10 0
docs 6 0
interop-pydantic-ai 9 0
interop 13 0
interop-langchain 9 0
interop-crewai 9 0
websockets 9 0
long-context 3 0
agent-eval 1 0
cohere 15 0
teachable 4 0
lmm 4 0
gpt-assistant-agent 3 0
cerebras 15 0
llama-index-agent 3 0
websurfer 15 0
retrievechat-couchbase 3 0
together 14 0
anthropic 16 0
mistral 14 0
groq 14 0
swarm 14 0
gemini 15 0
ollama 15 0
bedrock 15 0
core-llm 9 0
integration 12 0
falkordb 2 0
gemini-realtime 1 0
openai-realtime 1 0
autobuild 1 0
neo4j 2 0
deepseek 1 0
openai 1 0
Files with missing lines Coverage Δ
autogen/tools/dependency_injection.py 83.80% <50.00%> (-9.33%) ⬇️

... and 65 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.

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.

[Bug]: issubclass checking error in dependency_injection

3 participants