Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

False negative for E12 #108

Closed
myint opened this Issue · 2 comments

2 participants

@myint

I think we should get one of the E12 errors on the following input.


def long_function_name(
    var_one, var_two, var_three,
        var_four):
    print(var_one)
@florentx
Collaborator

This is explicitly accepted today, to give some flexibility.
I don't know if it should be more strict

This one is rejected:

def long_function_name(
    var_one, var_two, var_three,
    var_four, var_five, var_six,
    var_seven, var_eight, var_nine,
    var_last=some_thing):
    print(var_one)

And all the following are currently accepted:

def long_function_name(
    var_one, var_two, var_three,
    var_four, var_five, var_six,
    var_seven, var_eight, var_nine,
        var_last=some_thing):
    print(var_one)
def long_function_name(
    var_one, var_two, var_three,
    var_four, var_five, var_six,
        var_seven, var_eight, var_nine,
        var_last=some_thing):
    print(var_one)
def long_function_name(
    var_one, var_two, var_three,
        var_four, var_five, var_six,
        var_seven, var_eight, var_nine,
        var_last=some_thing):
    print(var_one)
def long_function_name(
        var_one, var_two, var_three,
        var_four, var_five, var_six,
        var_seven, var_eight, var_nine,
        var_last=some_thing):
    print(var_one)
@florentx
Collaborator

Not a bug.

@florentx florentx closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.