Skip to content

[1.0] Fix build: remove callSite from OpenVSX verification request#263

Merged
sachinh-amazon merged 1 commit into
1.0from
fix/openvsx-callsite-1.0
Jun 24, 2026
Merged

[1.0] Fix build: remove callSite from OpenVSX verification request#263
sachinh-amazon merged 1 commit into
1.0from
fix/openvsx-callsite-1.0

Conversation

@sachinh-amazon

Copy link
Copy Markdown
Contributor

Removes callSite property from the request object in add-openvsx-verification-check.diff. This property doesn't exist on IRequestOptions in Code-OSS 1.101, causing TS6133 compilation failure. The verification logic is unchanged.

@sachinh-amazon sachinh-amazon requested a review from a team as a code owner June 24, 2026 15:31
+++ code-editor-src/src/vs/platform/extensionManagement/common/extensionGalleryService.ts
@@ -680,6 +680,7 @@ export abstract class AbstractExtensionG
--- a/src/vs/platform/extensionManagement/common/extensionGalleryService.ts
+++ b/src/vs/platform/extensionManagement/common/extensionGalleryService.ts

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.

Shouldn't this be code-editor

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It wouldn't matter but let me change it for consistency with other patches. Good catch!

callSite property does not exist on IRequestOptions in Code-OSS 1.101.
This was causing a TypeScript compilation error on the 1.0 branch.
@sachinh-amazon sachinh-amazon force-pushed the fix/openvsx-callsite-1.0 branch from 6510616 to 43c5e68 Compare June 24, 2026 16:16
@sachinh-amazon sachinh-amazon added this pull request to the merge queue Jun 24, 2026
Merged via the queue into 1.0 with commit 5286506 Jun 24, 2026
1 check passed
@sachinh-amazon sachinh-amazon deleted the fix/openvsx-callsite-1.0 branch June 24, 2026 16:19
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.

2 participants