W503: yapf inserts line break before rather than after a '-' binary operator

[chtseac]

There is no option that I can find for this, unlike #38 and #252.

Code:

import inspect


def get_default_of(func, argname):
    argspec_of_func = inspect.getargspec(func)
    return argspec_of_func.defaults[argspec_of_func.args.index(argname) - len(argspec_of_func.args)]

Result:

import inspect


def get_default_of(func, argname):
    argspec_of_func = inspect.getargspec(func)
    return argspec_of_func.defaults[argspec_of_func.args.index(argname)
                                    - len(argspec_of_func.args)]

This yields W503 for pyflakes: line break before binary operator.

[AWhetter]

A line break should be before a binary operator. PEP8 was updated to expect binary operators before a line break (https://www.python.org/dev/peps/pep-0008/#should-a-line-break-before-or-after-a-binary-operator) and the flake8 warning has since been turned off by default (PyCQA/pycodestyle#499).

[ssbarnea]

Am I wrong to assume that this means that current version of yapf is incompatible with current version of pycodestyle? Is there a way to tune yapf to avoid creating files that do not match current pep-0008 requirements?

This is a serious issue as recently flake8 got an new release which finally fully switched to pycodestyle, so anyone that uses flake8 will have to update their code to match the new official style. Those trying to use yapf seems to be forced to either add ignore for W504 and W504 or... to drop using yapf.

[AWhetter]

They should be compatible. I think this issue can be closed because yapf has correct behaviour that matches PEP8. Also pycodestyle has turned off W503 by default and turned on W504 instead, and so yapf is compatible with pycodestyle.