Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 8, 2025

  • Understand the issue and required changes
  • Analyze how other commands use IDisplayProvider
  • Identify the pattern for displaying command suggestions
  • Add IDisplayProvider to MoveBranchCommandHandler constructor
  • Replace logger.SuggestStackUpdate() with displayProvider.DisplayMessage()
  • Remove unused SuggestStackUpdate logger extension method
  • Update all test files to include IDisplayProvider
  • Test the changes - all 208 tests pass

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] Add branch move command Add branch move command to move branches within a stack Sep 8, 2025
Copilot AI requested a review from geofflamrock September 8, 2025 11:17
stackConfig.Save(stackData);

logger.BranchMovedInStack(branchName, stack.Name);
logger.SuggestStackUpdate();
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot change this to use IDisplayProvider instead, like the way that examples are done for the status command.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to use IDisplayProvider.DisplayMessage() instead of the logger extension method. The suggestion message now uses proper styling with Example() formatting for the command names. Commit: 3d63a60

Copilot AI requested a review from geofflamrock September 9, 2025 10:10
@geofflamrock geofflamrock linked an issue Sep 11, 2025 that may be closed by this pull request
@geofflamrock geofflamrock force-pushed the copilot/fix-364 branch 2 times, most recently from 4dbff39 to 0ac1542 Compare September 25, 2025 07:16
@geofflamrock geofflamrock marked this pull request as ready for review September 25, 2025 07:16
Copilot AI and others added 9 commits September 29, 2025 19:31
…e-parenting

Co-authored-by: geofflamrock <2915931+geofflamrock@users.noreply.github.com>
…cumentation

Co-authored-by: geofflamrock <2915931+geofflamrock@users.noreply.github.com>
Co-authored-by: geofflamrock <2915931+geofflamrock@users.noreply.github.com>
@geofflamrock geofflamrock changed the title Add branch move command to move branches within a stack Add branch move command Sep 29, 2025
@geofflamrock geofflamrock merged commit 58b1d2f into main Sep 29, 2025
16 checks passed
@geofflamrock geofflamrock deleted the copilot/fix-364 branch September 29, 2025 09:47
@geofflamrock geofflamrock added the enhancement New feature or request label Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add branch move command

2 participants