|
I think that there are definite readability benefits to yapf==0.24.0. e.g.
|
left 0.24.0, right 0.21.0
|
2025,2028c2003,2005
|
< program_options.add_option(
|
< "--logLevel", dest="log_level", choices=log_levels,
|
< help="The log level. Accepted values are: {}."
|
< " [default: '%default'].".format(log_levels), default="info")
|
---
|
> program_options.add_option("--logLevel", dest="log_level", choices=log_levels,
|
> help="The log level. Accepted values are: {}."
|
> " [default: '%default'].".format(log_levels), default="info")
|
2260,2262c2237,2239
|
< ssh_connection_options = "{} {}".format(
|
< default_ssh_connection_options,
|
< options.ssh_connection_options if options.ssh_connection_options else "")
|
---
|
> ssh_connection_options = "{} {}".format(default_ssh_connection_options,
|
> options.ssh_connection_options
|
> if options.ssh_connection_options else "")
|
That said, the changes to the python code are quite a few lines of trivial differences. It also probably means converting percent sign format lines to format function calls. Hence this is a separate ticket.
|