Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make sure the file exists before checking it #87

Closed
wants to merge 1 commit into from
Closed

Make sure the file exists before checking it #87

wants to merge 1 commit into from

Conversation

pypingou
Copy link

Small patch to check that a file exists before running pep8 on it.

This prevents pep8 whatever to throw an exception that the file does not exists

florentx added a commit that referenced this pull request Jun 26, 2012
@florentx florentx closed this Jun 26, 2012
smira pushed a commit to smira/pep8 that referenced this pull request Jul 25, 2012
* upstream/master: (169 commits)
  Fix false positive E401; issue PyCQA#104.
  Add tests for issue PyCQA#100.
  Fix false positive for continuation line indentation when line contains comments. Closes PyCQA#100
  Fix issue with bpython. Closes PyCQA#101
  Back to development.
  Fix crash, and release 1.3.3; issue PyCQA#98.
  Release version 1.3.2.
  Add E902 errors for IOErrors; issue PyCQA#87.
  Update README.
  A little bit of rewording.
  Revert to the pre-1.2 behavior: --show-pep8 does not imply --first; issue PyCQA#89.
  Fix errors on E121 and E124; issue PyCQA#92.
  Add changelog entries for issue PyCQA#86 and issue PyCQA#91.
  Fix the default configuration path on Windows. Issue PyCQA#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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants