Skip to content

fix: upgrade Docker fallback warning to DeprecationWarning - #8156

Open
MOHAMMED WASIM KHAN (wasim-builds) wants to merge 1 commit into
microsoft:mainfrom
wasim-builds:fix/7462-security-warning-enhancement
Open

MOHAMMED WASIM KHAN (wasim-builds) wants to merge 1 commit into
microsoft:mainfrom
wasim-builds:fix/7462-security-warning-enhancement

Conversation

@wasim-builds

Copy link
Copy Markdown

Summary

Related to #7462

Changed from UserWarning to DeprecationWarning to make the security warning more visible when Docker is not available.

Changes

  • Changed warning type from UserWarning to DeprecationWarning in create_default_code_executor fallback

Test plan

  • Existing tests still pass

Changed from UserWarning to DeprecationWarning to make the security
warning more visible when Docker is not available.

Related to microsoft#7462
Copilot AI lite review requested due to automatic review settings September 2, 2026 05:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@wasim-builds

Copy link
Copy Markdown
Author

Hello maintainers,

Checking in on this pull request (fix: upgrade Docker fallback warning to DeprecationWarning) for microsoft/autogen. Contributing around multi-agent orchestration, agent communication, and tool execution in AutoGen has been exciting, and I'd love to help out with additional improvements and features across microsoft/autogen.

I am available for freelance/contract projects, full-time opportunities, or joining the team as a regular contributor.

You're welcome to connect or view my profile and repositories at https://github.com/wasim-builds.

This branch has not been deployed

No deployments
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.

2 participants