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

[.Net] update GeminiChatAgent.cs #3608

Merged
merged 4 commits into from
Oct 28, 2024
Merged

Conversation

eltociear
Copy link
Collaborator

Why are these changes needed?

multipe -> multiple

Checks

@LittleLittleCloud
Copy link
Collaborator

@eltociear Hey, thanks for the PR, we are going through a major refactor on a new AutoGen architecture for the main branch currently so we may have to postpone merging this PR until then. You can find out more information for the new architecture under #3601.

@eltociear
Copy link
Collaborator Author

@LittleLittleCloud Thank you for contacting me! Noted!

@rysweet
Copy link
Collaborator

rysweet commented Oct 10, 2024

@LittleLittleCloud - can you decide whether this work should come forward and be adapted to 0.4?

@rysweet rysweet added the 0.2 Issues which are related to the pre 0.4 codebase label Oct 10, 2024
@rysweet
Copy link
Collaborator

rysweet commented Oct 10, 2024

This PR is against AutoGen 0.2. AutoGen 0.2 has been moved to the 0.2 branch. Please rebase your PR on the 0.2 branch or update it to work with the new AutoGen 0.4 that is now in main.

@rysweet rysweet added the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 10, 2024
@LittleLittleCloud LittleLittleCloud merged commit fda85e1 into microsoft:main Oct 28, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.2 Issues which are related to the pre 0.4 codebase awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants