False negative for E12 #108

Closed
myint opened this Issue Jul 24, 2012 · 2 comments

Projects

None yet

2 participants

@myint
Member
myint commented Jul 24, 2012

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 Dec 22, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment