Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

False report of E121 #92

Closed
gugu opened this Issue · 1 comment

2 participants

@gugu

This code throws E121 on line 7

fooff(aaaa,
      abbb,
      cca(
          vvv,
          aaa,
          dadd),
      "here is error: continuation line indentation is not a multiple of four",)

But it is just an arguments. When I change code to

fooff(aaaa,
      abbb,
      cca(vvv,
          aaa,
          dadd),
      "continuation line indentation is not a multiple of four",)

PEP8 does not show any error

@florentx
Collaborator

Thank you for this bug report; now fixed.

@florentx florentx closed this
@smira smira referenced this issue from a commit in smira/pep8
@smira smira Merge remote-tracking branch 'upstream/master'
* upstream/master: (169 commits)
  Fix false positive E401; issue #104.
  Add tests for issue #100.
  Fix false positive for continuation line indentation when line contains comments. Closes #100
  Fix issue with bpython. Closes #101
  Back to development.
  Fix crash, and release 1.3.3; issue #98.
  Release version 1.3.2.
  Add E902 errors for IOErrors; issue #87.
  Update README.
  A little bit of rewording.
  Revert to the pre-1.2 behavior: --show-pep8 does not imply --first; issue #89.
  Fix errors on E121 and E124; issue #92.
  Add changelog entries for issue #86 and issue #91.
  Fix the default configuration path on Windows. Issue #95.
  Back to development.
  Allow 'verbose' to be specified in the config file
  Clarify .pep can be located in any parent folder
  Show maximum length in "E501 line too long" error message
  Release version 1.3.1.
  Do not read the configuration when used as a module.
  ...

Conflicts:
	pep8.py
e8b0ecf
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.