Update Black tests #13375
Update Black tests #13375
Conversation
MichaReiser
added
internal
| -type something_that_is_long[ | ||
| - something_that_is_long = something_that_is_long | ||
| -] = something_that_is_long | ||
| +type something_that_is_long[something_that_is_long = something_that_is_long] = ( | ||
| + something_that_is_long | ||
| +) |
There was a problem hiding this comment.
I prefer Ruff's formatting here which is more in line with other assignment formatting where the preference is to break the right side before breaking the left.
| +def simple[T = something_that_is_long]( | ||
| + short1: int, short2: str, short3: bytes | ||
| +) -> float: | ||
| pass |
There was a problem hiding this comment.
I prefer Ruff's formatting here. Breaking the return type and arguments is more readable than breaking the type arguments (overall, prefer breaking the right before breaking the left)
|
|
||
| f"{1:{f'{2}'}}" | ||
| -f"{1:{f'{2}'}}" | ||
| +f'{1:{f'{2}'}}' |
| - case "test" if ( | ||
| - "any long condition" != "another long condition" and "this is a long condition" | ||
| - ): | ||
| + case "test" if "any long condition" != "another long condition" and "this is a long condition": |
There was a problem hiding this comment.
Related to new preview style that parenthesizes long if conditions.
| - | memoryview | ||
| - # comment with non-return annotation | ||
| - ), | ||
| + # comment with non-return annotation |
|
|
||
| class ClassC: | ||
| def f(self): | ||
| - |
There was a problem hiding this comment.
Ruff does not preserve empty lines at the beginning of a class (intentional decision)
| +def bobtwo(): | ||
| + # some comment here | ||
| def bobtwo(): | ||
| - |
There was a problem hiding this comment.
Ruff does not preserve empty lines between the class and any leading comment (intentional decision)
| +++ Ruff | ||
| @@ -1,5 +1,4 @@ | ||
| class Plotter: | ||
| - |
There was a problem hiding this comment.
Ruff does not preserve empty lines after a class (intentional deviation)
|
Summary
Pull in the latest black snapshot tests
Test Plan
cargo test. I reviewed the new snapshot tests