| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Previously, it was entirely plausible for us to remove something from
options.ignore overzealously. This is more confined logic and much more akin
to what I was intending the behaviour to be.
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Continuous Integration Updates
- Add `.travis.yml` for GitHub (first set of tests were run [here](https://travis-ci.org/PyCQA/flake8/builds/44403113))
- Update `tox.ini` to add
- `py25` environment
- `py{27,34}-flake8` environments
See merge request !12 (https://gitlab.com/pycqa/flake8/merge_requests/12)
|
| | | |
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Extra include eggs
Add `.eggs` and `*.egg` to `EXTRA_EXCLUDE`
This is because these are usually downloaded by setuptools and they're usually other people's code so you don't want to check them.
```
[marca@marca-mac2 requests-unixsocket]$ flake8
./.eggs/pbr-0.10.0-py2.7.egg/pbr/packaging.py:740:17: E265 block comment should start with '# '
./pbr-0.10.0-py3.4.egg/pbr/packaging.py:740:17: E265 block comment should start with '# '
```
See merge request !10
|
| | | | |
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Exit Flake8Command.run() cleanly if style checks pass
When using distutils/setuptools, it is common to perform several commands in sequence. For example: `python setup.py flake8 bdist_egg`. When `run()` exits by raising a `SystemExit` exception, it causes the entire chain to break - meaning flake8 cannot be used in combination with any other command.
This change simply alters the behavior to exit normally if there were no style violations found. Otherwise, it will exit immediately, as before, with a non-zero exit code.
See merge request !8
|
| | |/ |
|
| |\ \
| |/
|/|
| |
| |
| | |
Add first pass at integration style tests
See merge request !9
|
| |/
|
|
|
|
| |
In order to better prevent regressions (such as related to concurrency),
Add a integration test framework to simulate running flake8 with
arguments.
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Flush stdout
As noted in email, it seems that failing to flush stdout in the sub-processes means that some output might go missing. I've observed this in practice with Syntastic in Vim. Here I flush both stdout and stderr before sending the message back to the main process that will allow it to continue. This makes the problem disappear for me.
It occurs to me that if sub-processes are producing output independently, there is also be a risk that they will badly interleave their output. I haven't looked into this - my primary concern is invocation for a single file at a time.
See merge request !7
|
| |/
|
|
|
|
| |
Subprocesses write to stdout. They must flush their output before the
main process exits to avoid losing output or confusing the calling
program.
|
| |\
| |
| |
| |
| |
| | |
make flake8 pass in dirty flake8 root directory
See merge request !6
|
| |/
|
|
| |
Add exclude list to tox.ini
|
| |\
| |
| |
| |
| |
| | |
Add usedevelop to tox.ini
See merge request !5
|
| | |
| |
| |
| |
| |
| | |
Tox 1.6 has a feature to install code into the virtualenv via 'setup.py
develop' This makes it easier to source the tox virtualenv and test code
changes.
|
| | |
| |
| |
| |
| |
| | |
I have seen people who place the comment at the bottom of the file and it
would be backwards incompatible if we did anything but preserve it as a search
instead of match
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Only match '\s*# flake8[:=]\s*nqa' when it happens at the beginning of a
line. The previous version of this regex is incorrectly hitting the
following line:
# Any "# flake8: noqa" line?
Causing flake8 to not be run on engine.py.
Also fix flake8 issues in engine.py.
|
| |\ \
| |/
|/|
| |
| |
| | |
Add a .gitignore file
See merge request !3
|
| |/ |
|
| | |
|
| |
|
|
|
|
|
|
|
| |
In short, pep8's option processing would bomb out if there were neither args
provided or config files present (setup.cfg, tox.ini, .pep8, ...).
--install-hook should work regardless of whether a config file is present or
not, so we use a callback to add an argument to the parser's internal argument
list and skip the check altogether. Links for context are in the comments in
this diff.
|
| |
|
|
| |
Fixes #4 on GitLab
|
| | |
|
| |
|
|
|
| |
This preserves backwards compatibility in the event someone was using
EXTRA_IGNORE as it should have been used
|
| |\
| |
| |
| |
| |
| |
| |
| | |
Git hook file pattern match
I found that with the git hook installed, flake8 was failing to check some files. Upon examination, I found that the hook is coded to only consider files ending in '.py'. This works out to be an override of the expected behavior, which is to honor the "filename" option in tox.ini.
See merge request !1
|
| |/
|
|
|
|
| |
In a config file, a user can use the "filename" option to specify
a pattern for matching filenames. The git hook effectively ignores
this and only checks files ending in '.py'
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
^C was making the script hang (few scenarios possible).
This patch makes sure that the queue operations won't block forever.
Potentially related to #167
|
| |
|
|
| |
Fixes #165
|
| | |
|
| |
|
|
| |
Fixes #164
|
| |
|
|
| |
Fixed #166
|
| |\
| |
| |
| | |
Use unittest.mock if available
|
| |/
|
|
|
| |
Use unittest.mock https://docs.python.org/3/library/unitetst.mock.html
starting from >= Python 3.3
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |\ |
|
| | |\
| | |
| | |
| | | |
Allow to configure multiple jobs on Windows; issues #153 and #154. Closes #153, #154, #156
|
| | |/ |
|
| | | |
|
| | | |
|