Remove editor.formatOnType support

[brettcannon]

We keep coming across bugs with the editor.formatOnType setting due to not implementing it in Python itself. I think we need to take a hard look at whether the complexity if trying to make this work in TypeScript is just not worth the constant maintenance overhead. Otherwise we need to look at doing a language server in Python to make this work appropriately (if it's still worth it).

[DonJayamanne]

in TypeScript is just not worth the constant maintenance

Agreed, considering this will be thrown away.

[MikhailArkhipov]

Why would it be better in Python? ASTs cannot be used fully anyway b/c of performance impact - can't wait for parsing on enter. There is always partial parsing involved - with recovery since constructs may not be fully completed. Hence on type formatters typically have heuristics involved. It is not actually possible to have two vs code language servers unless you mean some custom out-of-proc execution like Jedi.

AFAICS there were 5 bugs filed over 3 months or so.

[brettcannon]

The key point is getting this right is hard, and all of this work is for a feature that doesn't seem to have ended up being critical to users based on the fact our team is finding the bugs and not the general public while affecting standard Python features (e.g. *args, relative imports, etc.). IOW I would rather free up your time for more important, impactful things rather than being forced to play whack-a-mole with the heuristics to try and get this right for all scenarios.

[brettcannon]

We are re-evaluating this decision later today.

[brettcannon]

We pushed the feature hard and ferreted out a bunch of bugs which are already mostly fixed, so we are going to keep the feature in place.