Merge pull request #919 from asottile/first_line_E302

Allow N-and-fewer blank lines before the first top level thing

Author: asottile

pycodestyle.py

@@ -350,7 +350,7 @@ def blank_lines(logical_line, blank_lines, indent_level, line_number,
     top_level_lines = BLANK_LINES_CONFIG['top_level']
     method_lines = BLANK_LINES_CONFIG['method']
 
-    if line_number < top_level_lines + 1 and not previous_logical:
+    if not previous_logical and blank_before < top_level_lines:
         return  # Don't expect blank lines before the first line
     if previous_logical.startswith('@'):
         if blank_lines:

testsuite/E30.py

@@ -16,11 +16,6 @@ def b():
 #:
 
 
-#: E302:3:1
-#!python
-# -*- coding: utf-8 -*-
-def a():
-    pass
 #: E302:2:1
 """Main module."""
 def _main():

testsuite/E30not.py

@@ -177,3 +177,13 @@ def foo():
     # for no E30x being emitted.
     def bar(): pass
     def baz(): pass
+#: Okay
+#!python
+# -*- coding: utf-8 -*-
+def a():
+    pass
+#: Okay
+def f(
+    a,
+):
+    pass

testsuite/test_blank_lines.py

@@ -139,7 +139,6 @@ class AFarEnoughClass(object):
     pass
 """)
         self.assertEqual([
-            'E302:4:1',  # some_function
             'E302:7:1',  # another_function
             'E302:14:1',  # SomeCloseClass
         ], result)
@@ -438,7 +437,7 @@ def some_function():
 
     def test_top_level_fewer_blank_lines(self):
         """
-        It will trigger an error when less 2 blank lines are found
+        It will trigger an error when less 3 blank lines are found
         before top level definitions.
         """
         result = self.check("""# First comment line.
@@ -471,7 +470,6 @@ class AFarEnoughClass(object):
     pass
 """)
         self.assertEqual([
-            'E302:5:1',  # some_function
             'E302:9:1',  # another_function
             'E302:17:1',  # SomeCloseClass
         ], result)