Repository navigation
On type formatting errors for args and kwargs #1257
Description
def notify(self, *args, **kwargs):- Use the above code
- Place cursor after the
: - Hit enter
def notify(self, * args, ** kwargs):Brett Cannon (@brettcannon) Personally I don't think we should be doing this. What do you think?
Activity
MikhailArkhipov commentedon Apr 2, 2018
Probably should not be doing this.
[-]On type formatting errors for args and kwards[/-][+]On type formatting errors for args and kwargs[/+]MisLink commentedon Apr 5, 2018
f-string has same issue.
before:
f'hello'after hit enter:
f 'hello'brettcannon commentedon Apr 5, 2018
Is there a way to run the formatter through some well-formatted code to see what changes it would make? Like could we run it against the stdlib and see a diff of what would shift to see if we're making any other errors?
MikhailArkhipov commentedon Apr 5, 2018
Write TS test and supply file line by line to
formatter.formatLineas insuite('Formatting - line formatter'Fix indentation on single-liners and f-strings in type formatting (#1312
MisLink commentedon Apr 9, 2018
I found another problem, it will break relative import:
before:
from . import somethingafter:
from.import somethingI tried development build, but it seems like it remove all formatting config.7 remaining items
Add telemetry reporting on VS analysis engine usage (#1382)
Jedi 0.12 (#1418)
Replace unzip package (#1419)
Fixes multiple issues with formatting on type (#1450)
Improve detection of the function argument list in autoformat (#1508)
PTVS engine update + handling of the interpreter change (#1613)
Changes to IntelliCode and engine downloads (#1715)
Fix bug, remove unnecessary flag, fix hash comparison on Linux (#1734)
Simplifies LS startup and update test baselines (#1762)
Rename engine to 'Microsoft Python Language Server' (#1768)
Format on type fixes (#1798)
Handle numbers formatted with underscores in tokenizer (#1819)
Improve detection if '=' is in function or lambda arguments (#1824)
Add command for loading of LS extensions (#1914)