Skip to content

Update MCP lockfile to address Component Governance finding - #2750

Merged
shaahji merged 1 commit into
mainfrom
shaahji/up_mcp
Oct 26, 2026
Merged

shaahji merged 1 commit into
mainfrom
shaahji/up_mcp

Conversation

@shaahji

@shaahji shaahji commented Oct 21, 2026

Copy link
Copy Markdown
Collaborator

Update MCP lockfile to address Component Governance finding

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Copilot AI balanced review requested due to automatic review settings October 21, 2026 21:47

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.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@xiaoyu-work

Copy link
Copy Markdown
Member

is this for our internal compliance only? but Olive is open sourced and users may need pypi feed?

@shaahji

shaahji commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator Author

is this for our internal compliance only? but Olive is open sourced and users may need pypi feed?

That's a good point. I just compared the updates and all web address for dependencies are now pointing to internal feed. With MS policy to not be able to download from pypi, I don't know how I can generate this file without the address pointing to internal feed. Do you know a way around it?

@shaahji

shaahji commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator Author

Resolved the references to internal feed by generating the file thru' public workflow automation. GitHub workflows don't use isolated network so the upgrade job can reach pypi.org and update the file using publicly accessible web links.

@shaahji
shaahji enabled auto-merge (squash) October 26, 2026 20:41
@shaahji
shaahji merged commit c063dd3 into main Oct 26, 2026
9 checks passed
@shaahji
shaahji deleted the shaahji/up_mcp branch October 26, 2026 21:49
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