⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

Copy link

Copilot AI commented Jan 16, 2026

Addresses review feedback on #4: updateAllMarketplaces() prints a summary message even when no GitHub marketplaces exist to update.

Changes

  • Add early return in updateAllMarketplaces() when githubMarketplaces.length === 0
  • Print "No marketplaces to update." matching pattern in listMarketplaces() and listEnabledPlugins()
  • Add test case for empty GitHub marketplace scenario
// Before: prints "Updated 0 marketplace(s)" when only directory marketplaces exist
// After: prints "No marketplaces to update." and returns early
if (githubMarketplaces.length === 0) {
  console.log("No marketplaces to update.");
  return;
}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] WIP address feedback on plugin/marketplace commands Add empty state handling to updateAllMarketplaces Jan 16, 2026
Copilot AI requested a review from scaryrawr January 16, 2026 01:29
@scaryrawr scaryrawr marked this pull request as ready for review January 16, 2026 01:33
@scaryrawr scaryrawr merged commit 22cb6f4 into plugin-commands Jan 16, 2026
1 check passed
@scaryrawr scaryrawr deleted the copilot/sub-pr-4-again branch January 16, 2026 01:33
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