Adding Multiple Skill Directories Simplification + Integration Tests readme changes

Open rogerbarreto opened this issue 2 years ago • 0 comments

It is a common scenario having to import multiple skills.

Adding this as a param array to the extension will reduce number of calls and improve cleanliness.

As part of this PR I also spotted missing in the Readme the recommendation on using UserSecrets for Integration Tests.

Changed the readme and example config files to use davinci-003 models as our current standard for integration tests.

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Apr 22 '23 13:04 rogerbarreto