Skip to content

Fix SPLIT_ARGUMENTS_WHEN_COMMA_TERMINATED for one-item named argument list#1160

Merged
bwendling merged 2 commits intogoogle:mainfrom
alexey-pelykh:fix/split_arguments_when_comma_terminated-single-named-argument
Oct 10, 2023
Merged

Fix SPLIT_ARGUMENTS_WHEN_COMMA_TERMINATED for one-item named argument list#1160
bwendling merged 2 commits intogoogle:mainfrom
alexey-pelykh:fix/split_arguments_when_comma_terminated-single-named-argument

Conversation

@alexey-pelykh
Copy link
Copy Markdown
Contributor

@alexey-pelykh alexey-pelykh commented Oct 6, 2023

by taking precedence over SPLIT_BEFORE_NAMED_ASSIGNS:

Turn

r = f0(a=1,)

into

r = f0(
    a=1,
)

@alexey-pelykh alexey-pelykh force-pushed the fix/split_arguments_when_comma_terminated-single-named-argument branch from 408e49d to b5597a0 Compare October 6, 2023 07:03
@alexey-pelykh
Copy link
Copy Markdown
Contributor Author

@bwendling @Spitfire1900 a follow-up to #1142, yet while tests aren't failing, I'm not sure that this shuffling of code location is correct

… lists by taking precedence over SPLIT_BEFORE_NAMED_ASSIGNS
@alexey-pelykh alexey-pelykh force-pushed the fix/split_arguments_when_comma_terminated-single-named-argument branch from b5597a0 to 98916b7 Compare October 6, 2023 07:05
@bwendling bwendling merged commit 8ec75c6 into google:main Oct 10, 2023
@alexey-pelykh alexey-pelykh deleted the fix/split_arguments_when_comma_terminated-single-named-argument branch October 11, 2023 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants