Skip to content

.Net: Include the operation in the KernelFunction.Metadata#9556

Merged
markwallace-microsoft merged 1 commit intomicrosoft:mainfrom
markwallace-microsoft:users/markwallace/issue_9496_1
Nov 5, 2024
Merged

.Net: Include the operation in the KernelFunction.Metadata#9556
markwallace-microsoft merged 1 commit intomicrosoft:mainfrom
markwallace-microsoft:users/markwallace/issue_9496_1

Conversation

@markwallace-microsoft
Copy link
Member

Motivation and Context

Update for #9496 to include the RestApiOperation in the KernelFunction.MetaData

Description

This will help us avoid having to add all RestApiOperation properties individually to metadata.

Contribution Checklist

@markwallace-microsoft markwallace-microsoft requested a review from a team as a code owner November 5, 2024 19:12
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Nov 5, 2024
@github-actions github-actions bot changed the title Include the operation in the KernelFunction.Metadata .Net: Include the operation in the KernelFunction.Metadata Nov 5, 2024
@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Nov 5, 2024
github-merge-queue bot pushed a commit that referenced this pull request Nov 5, 2024
### Motivation and Context

Update for #9496 to include the `RestApiOperation` in the
`KernelFunction.MetaData`

### Description

This will help us avoid having to add all `RestApiOperation` properties
individually to metadata.

### 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 😄
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 5, 2024
@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Nov 5, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 5, 2024
@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Nov 5, 2024
Merged via the queue into microsoft:main with commit c10723d Nov 5, 2024
@markwallace-microsoft markwallace-microsoft deleted the users/markwallace/issue_9496_1 branch November 5, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants