Skip to content

Skip the builtinCG for extensions without a repo#308268

Merged
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/continued-antlion
Apr 7, 2026
Merged

Skip the builtinCG for extensions without a repo#308268
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/continued-antlion

Conversation

@mjbvz
Copy link
Copy Markdown
Collaborator

@mjbvz mjbvz commented Apr 7, 2026

If an built-in extension does not have a repo, we assume that it's being maintained by another team

If an built-in extension does not have a repo, we assume that it's being maintained by another team
Copilot AI review requested due to automatic review settings April 7, 2026 17:17
@mjbvz mjbvz enabled auto-merge April 7, 2026 17:17
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 build-time “built-in extensions component governance (CG)” downloader to gracefully skip built-in extensions that don’t declare a source repository, under the assumption they’re maintained externally.

Changes:

  • Add an early return in downloadExtensionDetails when extension.repo is missing.
  • Emit a log line explaining which extension/version was skipped and why.
Show a summary per file
File Description
build/lib/builtInExtensionsCG.ts Skip CG download for extensions missing repo to avoid failing/attempting invalid GitHub raw content fetches.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@mjbvz mjbvz merged commit d5117ad into microsoft:main Apr 7, 2026
25 of 27 checks passed
@vs-code-engineering vs-code-engineering bot added this to the 1.116.0 milestone Apr 7, 2026
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