Bad comment indentation

[advance512]

        businessManager = Business(
            businessManagerId, api=defaultApi
        ).remote_read(
            fields=[
                Business.Field.id,
                Business.Field.name,
                Business.Field.link,
                # Business.Field.timezone_id,
            ]
        )

gets formatted to:

        businessManager = Business(
            businessManagerId, api=defaultApi
        ).remote_read(
            fields=[
                Business.Field.id,
                Business.Field.name,
                Business.Field.link,
    # Business.Field.timezone_id,
            ]
        )

Note the comment. This triggers "E122 continuation line missing indentation or outdented" from flake8.

[ritiek]

What yapf version are you using? Mine is yapf 0.17.0.

It changes the below input

# random function
def test():
    pass
# a test
if 3==3:
    if 4==4:
        businessManager = Business(
            businessManagerId, api=defaultApi
        ).remote_read(
            fields=[
                Business.Field.id,
                Business.Field.name,
                Business.Field.link,
                # Business.Field.timezone_id,
            ]
        )

to

# random function
def test():
    pass


# a test
if 3 == 3:
    if 4 == 4:
        businessManager = Business(
            businessManagerId, api=defaultApi).remote_read(
                fields=[
                    Business.Field.id,
                    Business.Field.name,
                    Business.Field.link,
                    # Business.Field.timezone_id,
                ])

which I believe is the expected formatting.

[bwendling]

I get the same as @ritiek running against ToT.

[woodmd]

I'm seeing the same issue as @advance512 using the latest version of yapf (0.22.0) and python 2.7.14. If I set the style on the command-line (e.g. with yapf --style=pep8) the formatting is as expected but when running without arguments I get wrong indentation for comments. I'm configuring my default style using a setup.cfg with the following:

[yapf]
based_on_style = pep8
spaces_before_comment = 4
split_before_logical_operator = true
column_limit = 100

[zheolong]

@advance512 same issue, my yapf version is 0.30.0