summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Prevent unintended behaviour modifying options.ignorefeature/register-optional-checksIan Cordasco2014-12-181-5/+7
| | | | | | 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.
* Add ability to turn checks off by defaultIan Cordasco2014-12-182-4/+35
|
* Merge branch 'feature/continuous-integration' into 'master'Ian Cordasco2014-12-182-1/+33
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | 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)
| * Add configuration so Travis will run on our mirrorIan Cordasco2014-12-171-0/+20
| |
| * Ensure flake8 passes itself in testsIan Cordasco2014-12-171-1/+13
| |
* | Merge branch 'extra_include_eggs' into 'master'Ian Cordasco2014-12-061-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | Add .eggs and *.egg to EXTRA_EXCLUDEMarc Abramowitz2014-11-251-1/+1
| | |
* | | Merge branch 'master' into 'master'Ian Cordasco2014-12-061-1/+2
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | 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
| * | Do not exit early if style checks passJosh2014-10-281-1/+2
| |/
* | Merge branch 'testing' into 'master'Ian Cordasco2014-11-141-0/+71
|\ \ | |/ |/| | | | | | | Add first pass at integration style tests See merge request !9
| * Add first pass at integration style testsJoe Gordon2014-10-311-0/+71
|/ | | | | | In order to better prevent regressions (such as related to concurrency), Add a integration test framework to simulate running flake8 with arguments.
* Prepare release 2.2.52.2.5Ian Cordasco2014-10-192-1/+8
|
* Merge branch 'flush_stdout' into 'master'Ian Cordasco2014-10-201-0/+3
|\ | | | | | | | | | | | | | | | | | | 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
| * Flush all subprocess output before exitWeeble2014-10-191-0/+3
|/ | | | | | Subprocesses write to stdout. They must flush their output before the main process exits to avoid losing output or confusing the calling program.
* Merge branch 'ignore' into 'master'Ian Cordasco2014-10-151-0/+1
|\ | | | | | | | | | | make flake8 pass in dirty flake8 root directory See merge request !6
| * Make flake8 pass in root directoryJoe Gordon2014-10-151-0/+1
|/ | | | Add exclude list to tox.ini
* Merge branch 'setupdevelop' into 'master'Ian Cordasco2014-10-151-0/+2
|\ | | | | | | | | | | Add usedevelop to tox.ini See merge request !5
| * Have tox install via setup.py developJoe Gordon2014-10-141-0/+2
| | | | | | | | | | | | 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.
* | Loosen the regular expression searchIan Cordasco2014-10-151-2/+2
| | | | | | | | | | | | 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
* | Make _flake8_noqa tighterJoe Gordon2014-10-151-7/+4
| | | | | | | | | | | | | | | | | | | | | | 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.
* | Merge branch 'git' into 'master'Ian Cordasco2014-10-151-0/+4
|\ \ | |/ |/| | | | | | | Add a .gitignore file See merge request !3
| * Add a .gitignore fileJoe Gordon2014-10-141-0/+4
|/
* Release v2.2.42.2.4Ian Cordasco2014-10-093-2/+23
|
* Make --install-hook work without config filesIan Cordasco2014-10-091-2/+16
| | | | | | | | | 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.
* Only initialize options.exclude if it is false-yIan Cordasco2014-10-091-1/+1
| | | | Fixes #4 on GitLab
* When run as a setuptools command, the previous fix breaks flake8Ian Cordasco2014-10-091-0/+6
|
* Fix GitLab #3 by actually excluding .toxIan Cordasco2014-10-092-1/+6
| | | | | This preserves backwards compatibility in the event someone was using EXTRA_IGNORE as it should have been used
* Merge branch 'git_hook-file_pattern_match' into 'master'Ian Cordasco2014-10-031-3/+8
|\ | | | | | | | | | | | | | | 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
| * Honor file pattern matching for git hookChris Buccella2014-09-231-3/+8
|/ | | | | | 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'
* Refactor Lukasz's workIan Cordasco2014-09-131-15/+22
|
* got rid of extra line added by accident.Joseph Kahn2014-09-131-1/+0
|
* Added my own extensions to the known extensions list, shameless self promotion.Joseph Kahn2014-09-131-0/+5
|
* Applied icordasc PR commentsLukasz Dobrzanski2014-09-131-5/+5
|
* Fix SIGINT when running on multiprocessingLukasz Dobrzanski2014-09-131-12/+27
| | | | | | | ^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
* Disable multiprocessing when using stdinIan Cordasco2014-09-123-4/+23
| | | | Fixes #165
* Move fix for #164Ian Cordasco2014-09-121-2/+2
|
* Disable --jobs on WindowsIan Cordasco2014-09-122-2/+8
| | | | Fixes #164
* Disable multiprocessing when using --diffIan Cordasco2014-09-111-0/+3
| | | | Fixed #166
* Merged in lukmdo/flake8/switch_use_unitest (pull request #48)Ian Cordasco2014-09-092-2/+17
|\ | | | | | | Use unittest.mock if available
| * Use unittest.mock if availableLukasz Dobrzanski2014-09-102-2/+17
|/ | | | | Use unittest.mock https://docs.python.org/3/library/unitetst.mock.html starting from >= Python 3.3
* Release v2.2.3Ian Cordasco2014-08-251-0/+1
|
* Release v2.2.32.2.3Ian Cordasco2014-08-252-1/+6
|
* Actually enable auto job configuration by defaultIan Cordasco2014-08-251-1/+1
|
* Release v2.2.2Ian Cordasco2014-07-041-0/+2
|
* Removed tag 2.2.22.2.2Ian Cordasco2014-07-041-0/+2
|
* Merge bitbucket remoteIan Cordasco2014-07-041-0/+10
|\
| * Merged in flox/flake8/mp-win32 (pull request #45)Ian Cordasco2014-07-021-0/+10
| |\ | | | | | | | | | Allow to configure multiple jobs on Windows; issues #153 and #154. Closes #153, #154, #156
| | * Allow to configure multiple jobs on Windows; issues #153 and #154Florent Xicluna2014-07-011-0/+10
| |/
* | Release v2.2.2Ian Cordasco2014-07-041-0/+1
| |
* | Bump flake8 version numberIan Cordasco2014-07-041-1/+1
| |