The Wayback Machine - https://web.archive.org/web/20201127073803/https://github.com/microsoft/vscode/issues/46678
Skip to content
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

Run organise imports on save #46678

Closed
jrieken opened this issue Mar 27, 2018 · 4 comments
Closed

Run organise imports on save #46678

jrieken opened this issue Mar 27, 2018 · 4 comments

Comments

@jrieken
Copy link
Member

@jrieken jrieken commented Mar 27, 2018

re #46589

It should be possible to configure organise import (read code actions in general) so that they run on save.

@mjbvz
Copy link
Contributor

@mjbvz mjbvz commented Mar 27, 2018

Yes, currently the feature is not reliable enough for me to feel confident triggering it on save but next iteration we should hook this up

@mjbvz
Copy link
Contributor

@mjbvz mjbvz commented Apr 16, 2018

Depends on #42092

@uloco
Copy link

@uloco uloco commented Apr 17, 2018

A better option imho would be to make it configurable to be run when formatting, rather than on save. At least I would expect auto format to handle such things, rather a file save...

@kieferrm kieferrm mentioned this issue Apr 17, 2018
39 of 45 tasks complete
@mjbvz
Copy link
Contributor

@mjbvz mjbvz commented Apr 23, 2018

Enabled by #48086,

	"editor.codeActionsOnSave": {
		"source.organizeImports": true
	}

This is not enable by default currently

@mjbvz mjbvz closed this Apr 23, 2018
@mjbvz mjbvz added the on-testplan label Apr 23, 2018
@mjbvz mjbvz mentioned this issue Apr 23, 2018
2 of 2 tasks complete
@vscodebot vscodebot bot locked and limited conversation to collaborators Jun 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.