Skip to content

Add quickstart for local dev/build. update documents - #436

Merged
Dinah Xiaoman G (DinahK-2SO) merged 14 commits into
mainfrom
user/DinahK-2SO/update-build-docs
Jul 20, 2026
Merged

Add quickstart for local dev/build. update documents#436
Dinah Xiaoman G (DinahK-2SO) merged 14 commits into
mainfrom
user/DinahK-2SO/update-build-docs

Conversation

@DinahK-2SO

@DinahK-2SO Dinah Xiaoman G (DinahK-2SO) commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Add quick start for local dev/build;

Document update: remove submodules instructions; update doc and script to accept vs2026

Copilot AI review requested due to automatic review settings July 17, 2026 08:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 build documentation to remove outdated submodule setup steps and adjusts the razzle.cmd MSBuild discovery logic to allow newer Visual Studio major versions (intended to include VS 18 / VS2026), while improving failure behavior with explicit non-zero exit codes.

Changes:

  • Remove “git submodule update” instructions from build docs and Copilot instructions.
  • Broaden vswhere MSBuild discovery to [17.0,19.0) and switch error paths to exit /b 1.
  • Add a guard to ensure the resolved MSBuild.exe path exists, and ensure the script exits 0 on success.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
tools/razzle.cmd Broaden VS/MSBuild detection to include newer VS major versions; improve error handling and add a resolution guard.
doc/building.md Remove submodule initialization instructions and state dependencies are restored via build tooling.
.github/copilot-instructions.md Remove submodule initialization instructions from Copilot contributor guidance.

Comment thread tools/razzle.cmd Outdated
Comment thread tools/razzle.cmd Outdated
Comment thread .github/copilot-instructions.md Outdated
Copilot AI review requested due to automatic review settings July 17, 2026 09:46
@DinahK-2SO Dinah Xiaoman G (DinahK-2SO) changed the title remove submodules instructions; update doc and script to accept vs2026 Add quickstart for local dev/build. update documents Jul 17, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Comment thread tools/razzle.cmd Outdated
Comment thread tools/razzle.cmd Outdated
Comment thread doc/quick-start-local-dev.md Outdated
Copilot AI review requested due to automatic review settings July 17, 2026 09:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Comment thread tools/razzle.cmd Outdated
Comment thread tools/razzle.cmd Outdated
Comment thread doc/quick-start-local-dev.md
Copilot AI review requested due to automatic review settings July 17, 2026 09:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Comment thread tools/razzle.cmd Outdated
Comment thread doc/quick-start-local-dev.md Outdated
Comment thread .github/copilot-instructions.md Outdated
Copilot AI review requested due to automatic review settings July 17, 2026 10:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread tools/razzle.cmd
Comment thread doc/quick-start-local-dev.md Outdated
Copilot AI review requested due to automatic review settings July 17, 2026 10:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread tools/razzle.cmd Outdated
Comment thread doc/quick-start-local-dev.md Outdated
@DinahK-2SO
Dinah Xiaoman G (DinahK-2SO) marked this pull request as draft July 17, 2026 16:51
@DinahK-2SO
Dinah Xiaoman G (DinahK-2SO) force-pushed the user/DinahK-2SO/update-build-docs branch from e95771d to d3ffd88 Compare July 20, 2026 07:03
@DinahK-2SO
Dinah Xiaoman G (DinahK-2SO) marked this pull request as ready for review July 20, 2026 07:10
Copilot AI review requested due to automatic review settings July 20, 2026 07:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread tools/razzle.cmd
Comment thread doc/quick-start-local-dev.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 20, 2026 07:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread doc/quick-start-local-dev.md Outdated
Copilot AI review requested due to automatic review settings July 20, 2026 07:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread tools/razzle.cmd Outdated
Copilot AI review requested due to automatic review settings July 20, 2026 08:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread tools/razzle.cmd Outdated
Comment thread tools/razzle.cmd Outdated
Copilot AI review requested due to automatic review settings July 20, 2026 08:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@DinahK-2SO
Dinah Xiaoman G (DinahK-2SO) merged commit ac07489 into main Jul 20, 2026
11 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.

3 participants