-
Notifications
You must be signed in to change notification settings - Fork 271
autofixers #1212
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
Comments
You should be able to configure an |
Cool for comparison in ale it looks like this,
|
Yeah, we could probably set something like that up, shouldn't be too hard. I think it was mostly a design decision, but I'm not opposed to the idea, I think it makes sense to make that easier for users considering how much everyone uses it. |
I can see merit in providing the functionality or forcing people to do it the native vim way, but I'd document it either way. I looked for it before asking but I may have missed it. |
Fwiw, adding the option would ultimately do it the same way, so it's not much of a big deal, it wouldn't be any different from any of the other options we provide already. Will see if I can add that at some point this week. |
For reference when adding the option, I think the two variants that work
or (with the extension)
The method you provided didn't work for me. |
@EvanCarroll See #1216 . As mentioned there, I think this needs a proper test ride, I think it should work, but there may be cases where the async nature of that call causes some issues. If you want to give that a try that'd be great. |
Describe the solution you'd like
The ability to have
:w
run a "fixer" before hand.Describe alternatives you've considered
If using using this project to replace ALE I can go to
Formatting
under the F5 tool and it'll run everything through rustfmt. I would like the ability to configure that to happen every time on:w
though.The text was updated successfully, but these errors were encountered: