Skip to content

Commit b446833

Browse files
authored
Merge pull request microsoft#180870 from microsoft/top-vicuna
remove nb.codeactiononsave setting, add `nb.experimental` to command ID
2 parents 89b615d + 91284cc commit b446833

File tree

3 files changed

+11
-11
lines changed

3 files changed

+11
-11
lines changed

src/vs/workbench/contrib/notebook/browser/contrib/saveParticipants/saveParticipants.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ class CodeActionOnSaveParticipant implements IStoredFileWorkingCopySaveParticipa
104104
return undefined;
105105
}
106106

107-
const setting = this.configurationService.getValue<{ [kind: string]: boolean } | string[]>('notebook.codeActionsOnSave');
107+
const setting = this.configurationService.getValue<{ [kind: string]: boolean } | string[]>('notebook.experimental.codeActionsOnSave');
108108
if (!setting) {
109109
return undefined;
110110
}

src/vs/workbench/contrib/notebook/browser/notebook.contribution.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -920,15 +920,15 @@ configurationRegistry.registerConfiguration({
920920
tags: ['notebookLayout'],
921921
default: false
922922
},
923-
[NotebookSetting.codeActionsOnSave]: {
924-
markdownDescription: nls.localize('notebook.codeActionsOnSave', "Experimental. Run a series of CodeActions for a notebook on save. CodeActions must be specified, the file must not be saved after delay, and the editor must not be shutting down. Example: `notebook.format: true`"),
925-
type: 'object',
926-
additionalProperties: {
927-
type: 'boolean'
928-
},
929-
tags: ['notebookLayout'],
930-
default: {}
931-
},
923+
// [NotebookSetting.codeActionsOnSave]: {
924+
// markdownDescription: nls.localize('notebook.codeActionsOnSave', "Experimental. Run a series of CodeActions for a notebook on save. CodeActions must be specified, the file must not be saved after delay, and the editor must not be shutting down. Example: `notebook.format: true`"),
925+
// type: 'object',
926+
// additionalProperties: {
927+
// type: 'boolean'
928+
// },
929+
// tags: ['notebookLayout'],
930+
// default: {}
931+
// },
932932
[NotebookSetting.confirmDeleteRunningCell]: {
933933
markdownDescription: nls.localize('notebook.confirmDeleteRunningCell', "Control whether a confirmation prompt is required to delete a running cell."),
934934
type: 'boolean',

src/vs/workbench/contrib/notebook/common/notebookCommon.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -937,7 +937,7 @@ export const NotebookSetting = {
937937
outputScrolling: 'notebook.output.scrolling',
938938
textOutputLineLimit: 'notebook.output.textLineLimit',
939939
formatOnSave: 'notebook.formatOnSave.enabled',
940-
codeActionsOnSave: 'notebook.codeActionsOnSave',
940+
codeActionsOnSave: 'notebook.experimental.codeActionsOnSave',
941941
outputWordWrap: 'notebook.output.wordWrap',
942942
outputLineHeightDeprecated: 'notebook.outputLineHeight',
943943
outputLineHeight: 'notebook.output.lineHeight',

0 commit comments

Comments
 (0)