Skip to content

ADR Updated Completion Service Selection Strategy - #5479

Merged
Mark Wallace (markwallace-microsoft) merged 3 commits into
microsoft:mainfrom
markwallace-microsoft:users/markwallace/adr-0038
Mar 19, 2024
Merged

Mark Wallace (markwallace-microsoft) merged 3 commits into
microsoft:mainfrom
markwallace-microsoft:users/markwallace/adr-0038

Conversation

@markwallace-microsoft

Copy link
Copy Markdown
Contributor

Motivation and Context

Description

Contribution Checklist

Merged via the queue into microsoft:main with commit b302550 Mar 19, 2024
@markwallace-microsoft
Mark Wallace (markwallace-microsoft) deleted the users/markwallace/adr-0038 branch March 19, 2024 14:09
LudoCorporateShark pushed a commit to LudoCorporateShark/semantic-kernel that referenced this pull request Aug 25, 2024
### Motivation and Context

<!-- Thank you for your contribution to the semantic-kernel repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->

### Description

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄

---------

Co-authored-by: Dmytro Struk <[email protected]>
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.

4 participants