-
Notifications
You must be signed in to change notification settings - Fork 658
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor webapi directory structure #100
Conversation
this commit moves any `*.cs` code living out of `webapi/CopilotChat`, into the `webapi/CopilotChat` folder, along with the `.csproj` file. other changes: - moves `deploy` folder into `scripts` folder - moves `importdocument` folder into `tools` folder - deletes the `CopilotChat.sln` file living at the root level
ee84db8
to
5554423
Compare
5554423
to
b8cf65e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice structureal update. Love the shorter namespaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove those credentials!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good
Motivation and Context
as the project has grown, the structure of webapi has become a bit hard to parse through, especially for a new dev coming into the project. this PR aims to reduce that tension and in general, reduce the clutter within the api.
Description
deploy
folder intoscripts
folder.importdocument
folder intotools
folder.webapi/CopilotChat
up a level and removes duplicate folders.SemanticKernel.Service
toCopilotChat.WebApi
.Contribution Checklist