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

[Copilot] Azure OpenAI Tools support added to our Copilot Toolkit #578

Merged
merged 8 commits into from
Feb 15, 2024

Conversation

qasimikram
Copy link
Contributor

@qasimikram qasimikram commented Feb 13, 2024

Summary

This pull request introduces Azure OpenAI Tools to our Copilot Toolkit. Currently, this functionality is open for application by Microsoft only. We will open this for community when next GA version of Azure Open AI is available which supports tools.

Work Item(s)

AB#487896

@qasimikram qasimikram requested a review from a team as a code owner February 13, 2024 23:34
@qasimikram qasimikram self-assigned this Feb 13, 2024
@github-actions github-actions bot added this to the Version 24.0 milestone Feb 14, 2024
Copy link
Contributor

@Groenbech96 Groenbech96 left a comment

Choose a reason for hiding this comment

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

Fix minor, but LGTM

Copy link
Contributor

@darjoo darjoo left a comment

Choose a reason for hiding this comment

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

Additionally, no test for any of it? :(

@qasimikram qasimikram requested a review from a team as a code owner February 14, 2024 11:43
@qasimikram
Copy link
Contributor Author

Additionally, no test for any of it? :(

Tests added.

@qasimikram qasimikram requested a review from darjoo February 14, 2024 11:49
@qasimikram
Copy link
Contributor Author

Fix minor, but LGTM

Done.

@qasimikram qasimikram dismissed Groenbech96’s stale review February 14, 2024 14:48

Done what was required.

darjoo
darjoo previously approved these changes Feb 15, 2024
darjoo
darjoo previously approved these changes Feb 15, 2024
Groenbech96
Groenbech96 previously approved these changes Feb 15, 2024
@qasimikram qasimikram enabled auto-merge (squash) February 15, 2024 12:15
@qasimikram qasimikram dismissed stale reviews from Groenbech96 and darjoo via fa66687 February 15, 2024 13:05
@qasimikram qasimikram requested a review from darjoo February 15, 2024 13:10
@qasimikram qasimikram merged commit 9bc7866 into main Feb 15, 2024
25 checks passed
@qasimikram qasimikram deleted the features/aoai-tools branch February 15, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants