-
Notifications
You must be signed in to change notification settings - Fork 28.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: improve settings descriptions for actions triggered on save #230052
base: main
Are you sure you want to change the base?
Conversation
@microsoft-github-policy-service agree |
5943ca9
to
22bef86
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also take a look at editor.codeActions.triggerOnFocusChange
, which will save in certain scenarios:
- auto save is
afterDelay
- editor code actions on save are set to
always
. - focus is changed
note that this only works atm for editor code actions on save, and not notebooks.
@@ -64,7 +64,7 @@ const codeActionsOnSaveSchema: IConfigurationPropertySchema = { | |||
items: { type: 'string' } | |||
} | |||
], | |||
markdownDescription: nls.localize('editor.codeActionsOnSave', 'Run Code Actions for the editor on save. Code Actions must be specified and the editor must not be shutting down. Example: `"source.organizeImports": "explicit" `'), | |||
markdownDescription: nls.localize('editor.codeActionsOnSave', 'Run Code Actions for the editor on save. Code Actions must be specified and the editor must not be shutting down. When [auto save](https://code.visualstudio.com/docs/editor/codebasics#_save-auto-save) is set to `afterDelay`, Code Actions will run when the file is saved explicitly, not when saved automatically. Example: `"source.organizeImports": "explicit" `'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for this and all cases below, instead of pointing towards documentation, please point it towards the auto save setting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: change to ....will only be run when the file is saved explicitly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
22bef86
to
2de9ac0
Compare
Not sure what you mean. This setting does not "save" in certain scenarios, it only triggers the code actions under some conditions, one of them being |
closes #230051