"Wrong Hanging Indentation" false positive?

[pylint-bot]

Originally reported by: Sean Wallitsch (BitBucket: shidarin, GitHub: @shidarin?)

---

Since 1.2.1, pylint no longer lets the following be an acceptable line continuation:

#!python
    parser.add_argument(
        "-o",
        "--output",
        help="specify the filetype to convert to, comma separated lists are "
             "accepted. Defaults to a .cc XML. Supported output formats are: "
             "{outputs}".format(outputs=str(OUTPUT_FORMATS.keys()))
    )

It's throwing C0330(bad-continuation) with the note "Wrong Hanging indentation." Seeming to prefer the following:

#!python
    parser.add_argument(
        "-o",
        "--output",
        help="specify the filetype to convert to, comma separated lists are "
        "accepted. Defaults to a .cc XML. Supported output formats are: "
        "{outputs}".format(outputs=str(OUTPUT_FORMATS.keys()))
    )

Which doesn't seem right. This looks like it was introduced in this commit, and we noticed it when the new version was being used on landscape.io (see github issue here). Is this a bug?

---

• Bitbucket: https://bitbucket.org/logilab/pylint/issue/232

[pylint-bot]

Original comment by Sean Wallitsch (BitBucket: shidarin, GitHub: @shidarin?):

---

I went ahead and created some sample code on a github gist.

[pylint-bot]

Original comment by Steven Myint (BitBucket: myint, GitHub: @myint?):

---

More examples of okay style: https://github.com/jcrocholl/pep8/blob/master/testsuite/E12not.py

And examples of actual bad continuations: https://github.com/jcrocholl/pep8/blob/master/testsuite/E12.py

[pylint-bot]

Original comment by Torsten Marek (BitBucket: tmarek, GitHub: @tmarek?):

---

This is fixed in e3ecc89.

[pylint-bot]

Original comment by Steven Myint (BitBucket: myint, GitHub: @myint?):

---

Thanks.