docs: clarify marketplace registration behavior - #2799
docs: clarify marketplace registration behavior#2799Dhruva Kumar (dhruvatr) wants to merge 2 commits into
Conversation
Bill Berry (WilliamBerryiii)
left a comment
There was a problem hiding this comment.
Thanks for improving the marketplace note. The comprehensive review found five items:
- Complete the PR description with the purpose, scope, and validation performed.
- Select the Documentation update change type.
- Resolve the currently blocked merge requirement before merging.
- Distinguish the known switching guidance from version-dependent duplicate-registration behavior (inline comment).
- Restore the explicit blockquote marker on the wrapped line (inline comment).
The change is documentation-only and has no functional, security, or accessibility regression. The recommended disposition is approve with comments after these documentation and readiness items are addressed.
| > The behavior when switching or duplicating same-name marketplace registrations | ||
| has not yet been observed. See [CLI Plugins](docs/getting-started/methods/cli-plugins.md) | ||
| > for details. |
There was a problem hiding this comment.
Medium — distinguish switching guidance from duplicate-registration uncertainty
This sentence groups switching and duplicate registration together as wholly unobserved. The linked guide is more specific: switching can require removing and re-adding the marketplace, while duplicate same-name handling can vary by CLI version. Please preserve that distinction here so readers do not miss the actionable switching guidance.
Suggested wording:
| > The behavior when switching or duplicating same-name marketplace registrations | |
| has not yet been observed. See [CLI Plugins](docs/getting-started/methods/cli-plugins.md) | |
| > for details. | |
| > Switching registrations can require removing and re-adding the marketplace. | |
| > Duplicate same-name registration behavior can vary by Copilot CLI version. See | |
| > [CLI Plugins](docs/getting-started/methods/cli-plugins.md) for details. |
| > Behavior when switching or duplicating same-name marketplace registrations | ||
| > has not been observed. See [CLI Plugins](docs/getting-started/methods/cli-plugins.md) | ||
| > The behavior when switching or duplicating same-name marketplace registrations | ||
| has not yet been observed. See [CLI Plugins](docs/getting-started/methods/cli-plugins.md) |
There was a problem hiding this comment.
Low — preserve the explicit blockquote marker
This wrapped line relies on lazy blockquote continuation, unlike the surrounding lines. Please retain the explicit marker for consistent source structure and renderer-independent behavior.
| has not yet been observed. See [CLI Plugins](docs/getting-started/methods/cli-plugins.md) | |
| > has not yet been observed. See [CLI Plugins](docs/getting-started/methods/cli-plugins.md) |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2799 +/- ##
========================================
Coverage 83.08% 83.09%
========================================
Files 183 171 -12
Lines 33712 33994 +282
Branches 25 0 -25
========================================
+ Hits 28010 28246 +236
- Misses 5699 5748 +49
+ Partials 3 0 -3
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Pull Request
Description
Related Issue(s)
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
hve-builderand addressed all actionable findings.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md).github/skills/*/SKILL.md).github/hooks/*/*.json)evals/)Other:
.ps1,.sh,.py)Sample Prompts (for AI Artifact Contributions)
User Request:
Execution Flow:
Output Artifacts:
Success Indicators:
For detailed contribution requirements, see:
Testing
Checklist
Required Checks
AI Artifact Contributions
hve-builderreview mode to review contributionhve-builderreviewRequired Local Checks
The following local-safe validation commands must pass before merging:
npm run validate:localnpm run validate:docsnpm run spell-checknpm run lint:md-linksSecurity Considerations
Additional Notes
Description
Improved the grammar and clarity of the sentence describing marketplace registration behavior in the HVE Core documentation.
Changes made
This is a documentation-only change and does not affect functionality.