Skip to content

Commit 9cdb992

Browse files
authored
source control quickDiffProvider has become experimental, breaking existing usages (microsoft#176357) (microsoft#176500)
Part of microsoft#175888
1 parent 155b155 commit 9cdb992

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/vs/workbench/api/common/extHostSCM.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import { MarshalledId } from 'vs/base/common/marshallingIds';
2121
import { ThemeIcon } from 'vs/base/common/themables';
2222
import { IMarkdownString } from 'vs/base/common/htmlContent';
2323
import { MarkdownString } from 'vs/workbench/api/common/extHostTypeConverters';
24-
import { checkProposedApiEnabled } from 'vs/workbench/services/extensions/common/extensions';
24+
import { checkProposedApiEnabled, isProposedApiEnabled } from 'vs/workbench/services/extensions/common/extensions';
2525

2626
type ProviderHandle = number;
2727
type GroupHandle = number;
@@ -488,10 +488,11 @@ class ExtHostSourceControl implements vscode.SourceControl {
488488

489489
set quickDiffProvider(quickDiffProvider: vscode.QuickDiffProvider | undefined) {
490490
this._quickDiffProvider = quickDiffProvider;
491-
if (quickDiffProvider?.label) {
492-
checkProposedApiEnabled(this._extension, 'quickDiffProvider');
491+
let quickDiffLabel = undefined;
492+
if (isProposedApiEnabled(this._extension, 'quickDiffProvider')) {
493+
quickDiffLabel = quickDiffProvider?.label;
493494
}
494-
this.#proxy.$updateSourceControl(this.handle, { hasQuickDiffProvider: !!quickDiffProvider, quickDiffLabel: quickDiffProvider?.label });
495+
this.#proxy.$updateSourceControl(this.handle, { hasQuickDiffProvider: !!quickDiffProvider, quickDiffLabel });
495496
}
496497

497498
private _commitTemplate: string | undefined = undefined;

0 commit comments

Comments
 (0)