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

Do not save after ALERename and ALEOrganizeImports #2823

Merged
merged 4 commits into from
Apr 17, 2020

Conversation

jeremija
Copy link
Contributor

Closes #2811

@jeremija jeremija force-pushed the do-not-save-on-autoimport branch from 6a7461b to c8c142b Compare October 10, 2019 01:55
autoload/ale/fix.vim Outdated Show resolved Hide resolved
autoload/ale/code_action.vim Outdated Show resolved Hide resolved
@chaucerbao
Copy link
Contributor

Just wanted to check in on this PR, and whether it was ready to merge.
@jeremija @w0rp

@jeremija
Copy link
Contributor Author

I believe I've addressed all required changes. I've been using this branch since and it's working well for me, but I'm happy to address any issues if there are any.

@w0rp w0rp merged commit dc054f3 into dense-analysis:master Apr 17, 2020
@w0rp
Copy link
Member

w0rp commented Apr 17, 2020

I have merged this now. Sorry for the delay!

@jeremija
Copy link
Contributor Author

Thanks 👍

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.

Using ALEOrganizeImports and g:ale_completion_tsserver_autoimport writes the buffer unexpectedly
3 participants