search

ev3dev / ev3dev-lang-python

Pure python bindings for ev3dev
MIT License
425 stars 144 forks source link

format code via yapf #716

Closed dwalton76 closed 4 years ago

dwalton76 commented 4 years ago

Formatted via

dwalton@laptop:ev3dev-lang-python$ yapf --style .yapf.cfg --in-place --recursive . 
dwalton@laptop:ev3dev-lang-python$ 
dwalton@laptop:ev3dev-lang-python$ cat .yapf.cfg 
[style]
based_on_style = pep8
COLUMN_LIMIT = 120
dwalton@laptop:ev3dev-lang-python$
dwalton76 commented 4 years ago

This patch is ~4300 lines.

WasabiFan commented 4 years ago

It doesn't look like this includes the yapf config or the make targets like you added in the other PR. Was that intentional? I think we should have the configuration included.

dwalton76 commented 4 years ago

oops, config and make target have been added now

dwalton76 commented 4 years ago

@WasabiFan this one look ok to merge? We can do a pre-commit hook via another commit/issue.

WasabiFan commented 4 years ago

Yeah, this looks good to me. Hopefully no one was working on a big PR :laughing: