Skip to content

Commit

Permalink
Fix two build warnings (#703)
Browse files Browse the repository at this point in the history
### Motivation and Context

Fix two analyzer warnings issued during the build.
  • Loading branch information
stephentoub committed Apr 27, 2023
1 parent 602d731 commit 31de013
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ public async Task ItCanRunCrudOperationWithPlainTextPayloadSuccessfullyAsync()

var messageContent = this._httpMessageHandlerStub.RequestContent;
Assert.NotNull(messageContent);
Assert.True(messageContent.Any());
Assert.True(messageContent.Length != 0);

var payloadText = System.Text.Encoding.UTF8.GetString(messageContent, 0, messageContent.Length);
Assert.Equal("fake-input-value", payloadText);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ internal static IServiceCollection AddPlannerFactory(this IServiceCollection ser
services.AddScoped<PlannerFactoryAsync>(sp => async (IKernel kernel) =>
{
// Create a kernel for the planner with the same contexts as the chat's kernel but with only skills we want available to the planner.
IKernel plannerKernel = new Kernel(new SkillCollection(), kernel.PromptTemplateEngine, kernel.Memory, kernel.Config, kernel.Log);
var plannerKernel = new Kernel(new SkillCollection(), kernel.PromptTemplateEngine, kernel.Memory, kernel.Config, kernel.Log);
//
// Add skills to the planner here.
Expand Down

0 comments on commit 31de013

Please sign in to comment.