mirror of
https://github.com/astral-sh/ruff.git
synced 2025-09-28 21:05:08 +00:00
7f6cb9dfb5
## Summary This formats call expressions with magic trailing comma and parentheses behaviour but without call chaining ## Test Plan Lots of new test fixtures, including some that don't work yet
233 lines
4.3 KiB
Text
233 lines
4.3 KiB
Text
---
|
|
source: crates/ruff_python_formatter/tests/fixtures.rs
|
|
input_file: crates/ruff_python_formatter/resources/test/fixtures/black/simple_cases/function2.py
|
|
---
|
|
## Input
|
|
|
|
```py
|
|
def f(
|
|
a,
|
|
**kwargs,
|
|
) -> A:
|
|
with cache_dir():
|
|
if something:
|
|
result = (
|
|
CliRunner().invoke(black.main, [str(src1), str(src2), "--diff", "--check"])
|
|
)
|
|
limited.append(-limited.pop()) # negate top
|
|
return A(
|
|
very_long_argument_name1=very_long_value_for_the_argument,
|
|
very_long_argument_name2=-very.long.value.for_the_argument,
|
|
**kwargs,
|
|
)
|
|
def g():
|
|
"Docstring."
|
|
def inner():
|
|
pass
|
|
print("Inner defs should breathe a little.")
|
|
def h():
|
|
def inner():
|
|
pass
|
|
print("Inner defs should breathe a little.")
|
|
|
|
|
|
if os.name == "posix":
|
|
import termios
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
elif os.name == "nt":
|
|
try:
|
|
import msvcrt
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
|
|
except ImportError:
|
|
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
|
|
elif False:
|
|
|
|
class IHopeYouAreHavingALovelyDay:
|
|
def __call__(self):
|
|
print("i_should_be_followed_by_only_one_newline")
|
|
else:
|
|
|
|
def foo():
|
|
pass
|
|
|
|
with hmm_but_this_should_get_two_preceding_newlines():
|
|
pass
|
|
```
|
|
|
|
## Black Differences
|
|
|
|
```diff
|
|
--- Black
|
|
+++ Ruff
|
|
@@ -32,34 +32,20 @@
|
|
|
|
|
|
if os.name == "posix":
|
|
- import termios
|
|
+ NOT_YET_IMPLEMENTED_StmtImport
|
|
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
-
|
|
elif os.name == "nt":
|
|
- try:
|
|
- import msvcrt
|
|
-
|
|
- def i_should_be_followed_by_only_one_newline():
|
|
- pass
|
|
+ NOT_YET_IMPLEMENTED_StmtTry
|
|
|
|
- except ImportError:
|
|
-
|
|
- def i_should_be_followed_by_only_one_newline():
|
|
- pass
|
|
-
|
|
elif False:
|
|
-
|
|
class IHopeYouAreHavingALovelyDay:
|
|
def __call__(self):
|
|
print("i_should_be_followed_by_only_one_newline")
|
|
-
|
|
else:
|
|
-
|
|
def foo():
|
|
pass
|
|
-
|
|
|
|
with hmm_but_this_should_get_two_preceding_newlines():
|
|
pass
|
|
```
|
|
|
|
## Ruff Output
|
|
|
|
```py
|
|
def f(
|
|
a,
|
|
**kwargs,
|
|
) -> A:
|
|
with cache_dir():
|
|
if something:
|
|
result = CliRunner().invoke(
|
|
black.main, [str(src1), str(src2), "--diff", "--check"]
|
|
)
|
|
limited.append(-limited.pop()) # negate top
|
|
return A(
|
|
very_long_argument_name1=very_long_value_for_the_argument,
|
|
very_long_argument_name2=-very.long.value.for_the_argument,
|
|
**kwargs,
|
|
)
|
|
|
|
|
|
def g():
|
|
"Docstring."
|
|
|
|
def inner():
|
|
pass
|
|
|
|
print("Inner defs should breathe a little.")
|
|
|
|
|
|
def h():
|
|
def inner():
|
|
pass
|
|
|
|
print("Inner defs should breathe a little.")
|
|
|
|
|
|
if os.name == "posix":
|
|
NOT_YET_IMPLEMENTED_StmtImport
|
|
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
elif os.name == "nt":
|
|
NOT_YET_IMPLEMENTED_StmtTry
|
|
|
|
elif False:
|
|
class IHopeYouAreHavingALovelyDay:
|
|
def __call__(self):
|
|
print("i_should_be_followed_by_only_one_newline")
|
|
else:
|
|
def foo():
|
|
pass
|
|
|
|
with hmm_but_this_should_get_two_preceding_newlines():
|
|
pass
|
|
```
|
|
|
|
## Black Output
|
|
|
|
```py
|
|
def f(
|
|
a,
|
|
**kwargs,
|
|
) -> A:
|
|
with cache_dir():
|
|
if something:
|
|
result = CliRunner().invoke(
|
|
black.main, [str(src1), str(src2), "--diff", "--check"]
|
|
)
|
|
limited.append(-limited.pop()) # negate top
|
|
return A(
|
|
very_long_argument_name1=very_long_value_for_the_argument,
|
|
very_long_argument_name2=-very.long.value.for_the_argument,
|
|
**kwargs,
|
|
)
|
|
|
|
|
|
def g():
|
|
"Docstring."
|
|
|
|
def inner():
|
|
pass
|
|
|
|
print("Inner defs should breathe a little.")
|
|
|
|
|
|
def h():
|
|
def inner():
|
|
pass
|
|
|
|
print("Inner defs should breathe a little.")
|
|
|
|
|
|
if os.name == "posix":
|
|
import termios
|
|
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
|
|
elif os.name == "nt":
|
|
try:
|
|
import msvcrt
|
|
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
|
|
except ImportError:
|
|
|
|
def i_should_be_followed_by_only_one_newline():
|
|
pass
|
|
|
|
elif False:
|
|
|
|
class IHopeYouAreHavingALovelyDay:
|
|
def __call__(self):
|
|
print("i_should_be_followed_by_only_one_newline")
|
|
|
|
else:
|
|
|
|
def foo():
|
|
pass
|
|
|
|
|
|
with hmm_but_this_should_get_two_preceding_newlines():
|
|
pass
|
|
```
|
|
|
|
|