Skip to content

Add sections for SDK and CLI#667

Merged
baijumeswani merged 5 commits intomainfrom
natke/license
Apr 24, 2026
Merged

Add sections for SDK and CLI#667
baijumeswani merged 5 commits intomainfrom
natke/license

Conversation

@natke
Copy link
Copy Markdown
Collaborator

@natke natke commented Apr 23, 2026

Clarify license terms for CLI and SDK

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

vercel Bot commented Apr 23, 2026

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

Project Deployment Actions Updated (UTC)
foundry-local Ready Ready Preview, Comment Apr 24, 2026 6:44pm

Request Review

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 repository root LICENSE to explicitly separate licensing information for the Foundry Local SDK vs the Foundry Local CLI.

Changes:

  • Adds an MIT License section labeled for “FOUNDRY LOCAL SDK”.
  • Adds a “FOUNDRY LOCAL CLI” section header before the existing Microsoft Software License Terms text.

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

Comment thread LICENSE Outdated
Comment thread LICENSE Outdated
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 4 out of 4 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 LICENSE Outdated
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 4 out of 4 changed files in this pull request and generated 3 comments.


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

Comment thread LICENSE Outdated
Comment thread LICENSE
Comment thread LICENSE
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 5 out of 5 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (2)

licenses/deepseek.md:1

  • This file removal eliminates the in-repo license/notice text for the DeepSeek model, but the website’s model browser still maps “deepseek” licenses to an internal URL (/licenses/deepseek in www/src/routes/models/service.ts). With this file (and the licenses/ folder) gone, that link/route will be broken and users won’t have an accessible DeepSeek license notice. Consider keeping this notice somewhere that is actually served (e.g., add a SvelteKit /licenses/deepseek route or a www/static/licenses/deepseek page) and update the model license URL mapping accordingly.
    licenses/phi.md:1
  • This file removal eliminates the in-repo license text for Phi, but the website’s model browser still maps “phi” licenses to an internal URL (/licenses/phi in www/src/routes/models/service.ts). With this file (and the licenses/ folder) removed, that link/route will be broken and the Phi license text will no longer be published anywhere in this repo. Consider keeping the Phi license notice at a served path and updating the model license URL mapping accordingly.

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

Comment thread README.md
samuel100
samuel100 previously approved these changes Apr 24, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@baijumeswani baijumeswani merged commit c641118 into main Apr 24, 2026
47 checks passed
@baijumeswani baijumeswani deleted the natke/license branch April 24, 2026 23:45
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.

5 participants