chore: fix build for ADO pipeline runs - #331867
Conversation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
📬 CODENOTIFYThe following users are being notified based on files changed in this PR: Ladislau Szomoru (@lszomoru)Matched files:
|
There was a problem hiding this comment.
Pull request overview
Updates Azure Pipelines builds to retrieve required Git LFS assets and support Azure DevOps repository origins.
Changes:
- Adds GitHub/Azure DevOps-aware repository test assertions.
- Fetches Copilot and distro LFS assets during pipeline runs.
- Documents Copilot extension dependency boundaries.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
extensions/copilot/src/extension/completions-core/vscode-node/lib/src/prompt/test/repository.test.ts |
Supports Azure DevOps origins in repository tests. |
extensions/copilot/.github/copilot-instructions.md |
Documents extension dependency boundaries. |
build/azure-pipelines/distro/download-distro.yml |
Pulls distro LFS objects from the correct checkout. |
build/azure-pipelines/copilot/test-integration-steps.yml |
Pulls Copilot test-cache LFS objects. |
build/azure-pipelines/common/checkout.yml |
Persists checkout credentials for subsequent fetches. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
0739e10
|
New verification build for GitHub: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=466402&view=results |
|
I'll merge this on Monday. |
Mohammad javad Dianat (dianatofficial)
left a comment
There was a problem hiding this comment.
Looks good. Error handling is solid.
Verification build GitHub: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=466347&view=results
Verification build ADO: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=466348&view=results