Skip to content

Fix cglicenses.json#312255

Merged
benvillalobos merged 1 commit intomainfrom
bv/oss-fix3
Apr 23, 2026
Merged

Fix cglicenses.json#312255
benvillalobos merged 1 commit intomainfrom
bv/oss-fix3

Conversation

@benvillalobos
Copy link
Copy Markdown
Member

@benvillalobos benvillalobos commented Apr 23, 2026

The OSS tool was breaking when hitting a license URL.

This revealed other license errors, so I went ahead and fixed.

This will cause the next oss tool run to generate a new thirdpartylicenses.txt file

Copilot AI review requested due to automatic review settings April 23, 2026 23:12
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 cglicenses.json to make OSS licensing metadata more robust (avoiding license URL fetch failures) and to correct/complete license overrides discovered during that process.

Changes:

  • Inline full MIT license texts for several packages that previously only declared the license name.
  • Add an explicit BSD-2-Clause license text override for emitter-listener with justification comments.
  • Update gcp-metadata’s fullLicenseTextUri to a raw.githubusercontent.com URL to avoid tooling issues with GitHub “blob” pages.
Show a summary per file
File Description
cglicenses.json Completes/repairs license override entries and switches a license URL to a raw content endpoint to prevent OSS tooling failures.

Copilot's findings

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

@benvillalobos benvillalobos enabled auto-merge (squash) April 23, 2026 23:19
@benvillalobos benvillalobos merged commit 0496ea6 into main Apr 23, 2026
30 checks passed
@benvillalobos benvillalobos deleted the bv/oss-fix3 branch April 23, 2026 23:45
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 23, 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