Skip to content

updates js docs to reflect new package structure#604

Merged
samuel100 merged 3 commits intomainfrom
prathikrao/js-docs-update
Apr 7, 2026
Merged

updates js docs to reflect new package structure#604
samuel100 merged 3 commits intomainfrom
prathikrao/js-docs-update

Conversation

@prathikr
Copy link
Copy Markdown
Contributor

@prathikr prathikr commented Apr 7, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 7, 2026 06:00
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
foundry-local Ready Ready Preview, Comment Apr 7, 2026 10:02am

Request Review

baijumeswani
baijumeswani previously approved these changes Apr 7, 2026
View reviewed changes
Copilot AI reviewed Apr 7, 2026
View reviewed changes
Copy link
Copy Markdown
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

Updates the JS SDK README’s Windows/WinML installation guidance to reflect the new package-based distribution approach.

Changes:

  • Replaces the --winml install-flag guidance with installing the foundry-local-sdk-winml package.
  • Updates the surrounding WinML install text accordingly.

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

Comment thread sdk/js/README.md
@prathikr prathikr requested a review from Copilot April 7, 2026 09:51
Copilot AI reviewed Apr 7, 2026
View reviewed changes
Copy link
Copy Markdown
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

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

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

Comment thread sdk/js/README.md
Comment thread sdk/js/README.md
@prathikr
copilot feedback
dcd17d2 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@prathikr prathikr requested a review from Copilot April 7, 2026 10:02
Copilot AI reviewed Apr 7, 2026
View reviewed changes
Copy link
Copy Markdown
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

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

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

Comment thread sdk/js/README.md
@samuel100 samuel100 self-requested a review April 7, 2026 12:47
@samuel100 samuel100 merged commit f3a8039 into main Apr 7, 2026
47 checks passed
@samuel100 samuel100 deleted the prathikrao/js-docs-update branch April 7, 2026 12:49
baijumeswani pushed a commit that referenced this pull request Apr 8, 2026
@prathikr @baijumeswani
updates js docs to reflect new package structure (#604)
8b373f3 
Co-authored-by: Prathik Rao <prathikrao@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@samuel100 samuel100 samuel100 approved these changes

Copilot code review Copilot Copilot left review comments

@baijumeswani baijumeswani baijumeswani left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@prathikr @samuel100 @baijumeswani