Too many blank lines, expected %s

[Jonathan Jacobs]

As of revision 29 of trunk, this warning appears to be missing an interpolation value.

[Tristan Seligmann (mithrandi)]

It's supposed to be getting an interpolation value thanks to this regex:

        'E303': ('W9015', r'too many blank lines, expected \((\d+)\)'),

Unfortunately I don't know what is going wrong here.

[HawkOwl (hawkowl)]

Okay, so I've chucked up a branch to "fix" this.

'E303': ('W9015', r'too many blank lines, expected \((\d+)\)'),

^ that was incorrect, and should not have had ", expected" in it. This was only in the internal pep8 blank lines replacement, but that relies on monkey patching, which doesn't currently seem to work. This fixes the core problem in this ticket, though.

[HawkOwl (hawkowl)]

I've pushed up another update to the branch to fix this fully.

It now properly monkeypatches PEP8. It also gets rid of some old compat stuff, which is not really needed, since PEP8 1.0 onwards supports this fine.