Skip to content

.Net AudioContent class moved to the Microsoft.SemanticKernel namespace#5112

Merged
SergeyMenshykh merged 4 commits intomicrosoft:mainfrom
SergeyMenshykh:namespace-for-audio-content
Feb 22, 2024
Merged

.Net AudioContent class moved to the Microsoft.SemanticKernel namespace#5112
SergeyMenshykh merged 4 commits intomicrosoft:mainfrom
SergeyMenshykh:namespace-for-audio-content

Conversation

@SergeyMenshykh
Copy link
Member

The AudioContent class lives in the Microsoft.SemanticKernel.Contents namespace, while the other content classes reside in the Microsoft.SemanticKernel namespace. This PR moves the AudioContent class to the Microsoft.SemanticKernel namespace to align it with the other content classes.

…ce to align it with the other content classes.
@SergeyMenshykh SergeyMenshykh added the .NET Issue or Pull requests regarding .NET code label Feb 21, 2024
@SergeyMenshykh SergeyMenshykh self-assigned this Feb 21, 2024
@SergeyMenshykh SergeyMenshykh requested a review from a team as a code owner February 21, 2024 22:27
@markwallace-microsoft markwallace-microsoft added kernel Issues or pull requests impacting the core kernel kernel.core labels Feb 21, 2024
Copy link
Member

@dmytrostruk dmytrostruk left a comment

Choose a reason for hiding this comment

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

My mistake, thanks for fixing this. Pipelines are failing, since it's breaking change. I've added suppression files to proceed with merging.

@SergeyMenshykh
Copy link
Member Author

My mistake, thanks for fixing this. Pipelines are failing, since it's breaking change. I've added suppression files to proceed with merging.

No problem at all. We should have done a better job reviewing the PR. Thanks for adding the suppression.

@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue Feb 22, 2024
Merged via the queue into microsoft:main with commit cd463b8 Feb 22, 2024
@SergeyMenshykh SergeyMenshykh deleted the namespace-for-audio-content branch February 22, 2024 14:39
LudoCorporateShark pushed a commit to LudoCorporateShark/semantic-kernel that referenced this pull request Aug 25, 2024
…ce (microsoft#5112)

The AudioContent class lives in the Microsoft.SemanticKernel.Contents
namespace, while the other content classes reside in the
Microsoft.SemanticKernel namespace. This PR moves the AudioContent class
to the Microsoft.SemanticKernel namespace to align it with the other
content classes.

---------

Co-authored-by: Dmytro Struk <13853051+dmytrostruk@users.noreply.github.com>
Bryan-Roe pushed a commit to Bryan-Roe-ai/semantic-kernel that referenced this pull request Oct 6, 2024
…ce (microsoft#5112)

The AudioContent class lives in the Microsoft.SemanticKernel.Contents
namespace, while the other content classes reside in the
Microsoft.SemanticKernel namespace. This PR moves the AudioContent class
to the Microsoft.SemanticKernel namespace to align it with the other
content classes.

---------

Co-authored-by: Dmytro Struk <13853051+dmytrostruk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel.core kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants