google / google/yapf

Prefer split named arguments rather than lists

Open
#691 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
14k
Forks
904
PR merge metrics
No merged PRs in 30d

Description

```
[style]
based_on_style=chromium
split_before_named_assigns=False
```

When there are lists as arguments yapf seems to adopt a weird splitting:
```python
def fooooooooooooo(aka=['aaaaaaaa', 'aaaaaaa', 'aaaa', 'aa',
'a'], types=['double', 'bool'],
validator=some_looooooooooong_name):
pass

def fooooooooooooo(type01=aaaaaa, type02=['double'], type03=['double'],
aka=['aaaaaaaa', 'aaaaaaa', 'aaaa', 'aa',
'a'], aaaaa=['double', 'bool'],
validator=some_looooooooooong_name):
pass
```

instead I would prefer:
```python
def fooooooooooooo(aka=['aaaaaaaa', 'aaaaaaa', 'aaaa', 'aa', 'a'],
types=['double', 'bool'],
validator=some_looooooooooong_name):
pass

def fooooooooooooo(type01=aaaaaa, type02=['double'], type03=['double'],
aka=['aaaaaaaa', 'aaaaaaa', 'aaaa', 'aa', 'a'],
aaaaa=['double', 'bool'],
validator=some_looooooooooong_name):
pass
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.