Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 9, 2026

A user requested opening a new pull request to address review comments from a separate review thread. This response clarifies the limitation and asks for direction.

Context

The review thread referenced contains several issues:

  • Parameterless constructors calling base() instead of base(default) in BotTokenClient, UserTokenClient, and UserClient
  • Redundant Next(IContext<TActivity> context) overload in Context.cs
  • Performance concern with repeated OrderBy operations in Router.cs

Response

Replied to the comment explaining that new PRs cannot be created programmatically, and requested clarification on whether to:

  1. Apply changes to the current PR Account Linking Url: Combined Auth Flow #188, or
  2. Provide a change summary for manual PR creation

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update account linking URL for combined auth flow Clarify PR scope: Cannot create new PRs programmatically Jan 9, 2026
Copilot AI requested a review from rajan-chari January 9, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants