Skip to content

Remove unused Skill.cs file from KernelHttpServer#307

Merged
lemillermicrosoft merged 3 commits intomicrosoft:mainfrom
glahaye:Remove-Unused-File
Apr 4, 2023
Merged

Remove unused Skill.cs file from KernelHttpServer#307
lemillermicrosoft merged 3 commits intomicrosoft:mainfrom
glahaye:Remove-Unused-File

Conversation

@glahaye
Copy link
Contributor

@glahaye glahaye commented Apr 4, 2023

Motivation and Context

Skill.cs was not used at all

Description

Remove Skill.cs

Contribution Checklist

@glahaye glahaye added the PR: ready for review All feedback addressed, ready for reviews label Apr 4, 2023
Copy link
Member

@rogerbarreto rogerbarreto left a comment

Choose a reason for hiding this comment

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

LGTM

@lemillermicrosoft lemillermicrosoft enabled auto-merge (squash) April 4, 2023 15:36
@adrianwyatt adrianwyatt added PR: ready to merge PR has been approved by all reviewers, and is ready to merge. PR: main and removed PR: ready for review All feedback addressed, ready for reviews labels Apr 4, 2023
@lemillermicrosoft lemillermicrosoft merged commit cb53fde into microsoft:main Apr 4, 2023
@glahaye glahaye deleted the Remove-Unused-File branch April 5, 2023 01:10
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Motivation and Context
Skill.cs was not used at all

### Description
Remove Skill.cs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: ready to merge PR has been approved by all reviewers, and is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants