Skip to content

Commit

Permalink
fix more warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
alvesfabi committed Feb 23, 2024
1 parent 5cfbcb8 commit e1ea576
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion webapi/Plugins/Chat/ApiConnectorPlugin.cs
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ public ApiConnectorPlugin(string bearerToken, IHttpClientFactory clientFactory,
// Failed to get graph data: {graphResponse.StatusCode}.
[SKFunction]
[Description("Call a Graph API with the OData query and the Graph API Scopes based on user input")]
public async Task<string> CallGraphApiTasksAsync([Description("Url of the Graph API with the OData query to call")] string apiToCall, [Description("Comma separated value string with the Graph API Scopes needed to execute the call")] string graphScopes, CancellationToken cancellationToken = default(CancellationToken))
public async Task<string> CallGraphApiTasksAsync([Description("Url of the Graph API with the OData query to call")] string apiToCall, [Description("Comma separated value string with the Graph API Scopes needed to execute the call")] string graphScopes, CancellationToken cancellationToken = default)
{
if (string.IsNullOrEmpty(apiToCall))
{
Expand Down
3 changes: 0 additions & 3 deletions webapi/Plugins/Chat/CopilotChatPlanner.cs
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,8 @@ public class CopilotChatPlanner
/// Matches: $variableName, $variable_name, $variable-name, $some_variable_Name, $variableName123, $variableName_123, $variableName-123
/// Does not match: $123variableName, $100 $200
/// </summary>

// private const string VariableRegex = @"\$([A-Za-z]+[_-]*[\w]+)";
private const string VariableRegex = @"\$((?!top|filter|select|expand|orderby|skip|count|search)[A-Za-z0-9_]*)";


/// <summary>
/// Supplemental text to add to the plan goal if PlannerOptions.Type is set to Stepwise.
/// Helps the planner know when to bail out to request additional user input.
Expand Down

0 comments on commit e1ea576

Please sign in to comment.