Do not report E704 in the default configuration; issue #277

Author: florentx

CHANGES.txt

@@ -7,10 +7,11 @@ Changelog
 
 * Report E731 for lambda assignment. (Issue #277)
 
-* Report E704 for one-liner def instead of E701. (Issue #277)
+* Report E704 for one-liner def instead of E701.
+  Do not report this error in the default configuration. (Issue #277)
 
 * Replace codes E111, E112 and E113 with codes E114, E115 and E116
-  for wrong indentation of comments. (Issue #274)
+  for bad indentation of comments. (Issue #274)
 
 * Report E266 instead of E265 when the block comment starts with
   multiple ``#``. (Issue #270)

docs/intro.rst

@@ -313,7 +313,7 @@ This is the current list of error and warning codes:
 +----------+----------------------------------------------------------------------+
 | E703     | statement ends with a semicolon                                      |
 +----------+----------------------------------------------------------------------+
-| E704     | multiple statements on one line (def)                                |
+| E704 (*) | multiple statements on one line (def)                                |
 +----------+----------------------------------------------------------------------+
 | E711 (^) | comparison to None should be 'if cond is None:'                      |
 +----------+----------------------------------------------------------------------+
@@ -367,7 +367,7 @@ This is the current list of error and warning codes:
 
 
 **(*)** In the default configuration, the checks **E123**, **E133**, **E226**,
-**E241** and **E242** are ignored because they are not rules unanimously
+**E241**, **E242** and **E704** are ignored because they are not rules unanimously
 accepted, and `PEP 8`_ does not enforce them.  The check **E133** is mutually
 exclusive with check **E123**.  Use switch ``--hang-closing`` to report **E133**
 instead of **E123**.

pep8.py

@@ -64,7 +64,7 @@
     from ConfigParser import RawConfigParser
 
 DEFAULT_EXCLUDE = '.svn,CVS,.bzr,.hg,.git,__pycache__'
-DEFAULT_IGNORE = 'E123,E226,E24'
+DEFAULT_IGNORE = 'E123,E226,E24,E704'
 if sys.platform == 'win32':
     DEFAULT_CONFIG = os.path.expanduser(r'~\.pep8')
 else:

testsuite/test_api.py

@@ -178,7 +178,7 @@ def parse_argv(argstring):
 
         options = parse_argv('').options
         self.assertEqual(options.select, ())
-        self.assertEqual(options.ignore, ('E123', 'E226', 'E24'))
+        self.assertEqual(options.ignore, ('E123', 'E226', 'E24', 'E704'))
 
         options = parse_argv('--doctest').options
         self.assertEqual(options.select, ())