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

Skip to content

Fix agents and agents tests checks #1141

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

Jump to bottom
Merged
merged 3 commits into from
Feb 26, 2025
Merged

Fix agents and agents tests checks #1141

merged 3 commits into from
Feb 26, 2025

Conversation

@marklysze
Copy link
Collaborator

@marklysze marklysze commented Feb 26, 2025

Why are these changes needed?

After restoring autogen/agents and test/agents in pyproject.toml a number of mypy checks were failing. This corrects them.

Related issue number

N/A

Checks

@marklysze marklysze added bug Something isn't working testing labels Feb 26, 2025
@marklysze marklysze requested a review from davorrunje February 26, 2025 00:04
@marklysze marklysze self-assigned this Feb 26, 2025
@marklysze marklysze merged commit e3cf756 into main Feb 26, 2025
17 checks passed
@marklysze marklysze deleted the testfixes20250226 branch February 26, 2025 18:27
@github-project-automation github-project-automation bot moved this to Done in ag2 Feb 26, 2025
@codecov
Copy link

codecov bot commented Feb 26, 2025

Codecov Report

Attention: Patch coverage is 22.22222% with 21 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
autogen/agents/experimental/discord/discord.py 22.22% 7 Missing ⚠️
autogen/agents/experimental/slack/slack.py 22.22% 7 Missing ⚠️
autogen/agents/experimental/telegram/telegram.py 22.22% 7 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (0790aae) and HEAD (c67943b). Click for more details.

HEAD has 1273 uploads less than BASE
Flag BASE (0790aae) HEAD (c67943b)
core-without-llm 15 1
3.9 78 0
ubuntu-latest 147 1
3.13 88 0
macos-latest 108 0
3.11 67 1
3.10 97 0
commsagent-discord 9 0
optional-deps 139 0
3.12 39 0
browser-use 7 0
windows-latest 114 0
commsagent-slack 9 0
commsagent-telegram 9 0
crawl4ai 13 0
docs 6 0
graph-rag-falkor-db 6 0
interop 13 0
interop-crewai 9 0
interop-langchain 9 0
interop-pydantic-ai 9 0
jupyter-executor 9 0
twilio 9 0
retrievechat 10 0
retrievechat-pgvector 10 0
websockets 9 0
retrievechat-mongodb 10 0
retrievechat-qdrant 11 0
agent-eval 1 0
gpt-assistant-agent 3 0
lmm 4 0
teachable 4 0
gemini 15 0
websurfer 15 0
anthropic 16 0
bedrock 14 0
cerebras 14 0
llama-index-agent 3 0
retrievechat-couchbase 3 0
cohere 15 0
groq 14 0
ollama 14 0
mistral 14 0
together 14 0
swarm 14 0
reasoning 14 0
long-context 3 0
integration 12 0
falkordb 2 0
autobuild 1 0
core-llm 4 0
neo4j 2 0
deepseek 1 0
openai 1 0
Files with missing lines Coverage Δ
autogen/agents/experimental/websurfer/websurfer.py 50.00% <ø> (-38.89%) ⬇️
autogen/agents/experimental/discord/discord.py 42.85% <22.22%> (-50.90%) ⬇️
autogen/agents/experimental/slack/slack.py 42.85% <22.22%> (-50.90%) ⬇️
autogen/agents/experimental/telegram/telegram.py 42.85% <22.22%> (-50.90%) ⬇️

... and 59 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davorrunje davorrunje Awaiting requested review from davorrunje

Assignees

@marklysze marklysze

Labels

bug Something isn't working testing

Projects

ag2
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@marklysze