Skip to content

Conversation

@d10c
Copy link
Contributor

@d10c d10c commented Oct 27, 2025

Added clarifications based on my experience releasing the extension for the first time.

Added clarifications based on my experience releasing the extension for the first time.
@d10c d10c marked this pull request as ready for review October 27, 2025 16:49
@d10c d10c requested a review from a team as a code owner October 27, 2025 16:49
Copilot AI review requested due to automatic review settings October 27, 2025 16:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the release and testing documentation to include clarifications based on first-time release experience, making the documentation more accessible and complete for new contributors.

  • Added detailed instructions for configuring variant analysis repositories
  • Enhanced model pack testing steps with prerequisite settings and clearer directory guidance
  • Added note about required permissions for approving marketplace deployments

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/test-plan.md Added concrete UI instructions for variant analysis setup, VSCode settings prerequisite for model packs, and fixed numbering inconsistencies
docs/releasing.md Added clarification about required group membership for deployment approval

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@d10c d10c requested a review from nickrolfe October 27, 2025 16:51
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@d10c d10c enabled auto-merge October 27, 2025 17:02
@d10c d10c merged commit f2ccb79 into github:main Oct 27, 2025
38 of 40 checks passed
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