| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |
|
|
| |
found via `pre-commit try-repo https://github.com/codespell-project/codespell --all-files`
|
| |
|
|
|
| |
letter error codes). A single change in the regex `NOQA_INLINE_REGEXP` in `defaults.py` will allow to catch error codes which consist of more than one letter.
This will close #549.
|
| | |
|
| |
|
|
| |
This provides a step by step doc for the regular expression, and makes iterating on it much easier
|
| | |
|
| | |
|
| |
|
|
|
| |
Instead of just using Flake8 and pylint to keep Flake8 clean, let's also
use black to make it less manual for clean-up.
|
| |
|
|
|
|
|
|
|
|
|
| |
```
$ python3.7 -Werror
Python 3.7.0rc1 (default, Jun 16 2018, 03:32:08)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> '# noqa(?:: (?P<codes>([A-Z][0-9]+(?:[,\s]+)?)+))?'
File "<stdin>", line 1
SyntaxError: invalid escape sequence \s
```
|
| |
|
|
|
|
|
| |
To match our new configuration file format and its allowance for spaces
in the list
Closes #356
|
| |
|
|
|
|
| |
* Use set literals instead of set([...])
* Avoid list(sorted(...)) as sorted returns a list
* Replace dict() with dict literal
|
| |
|
|
|
|
|
|
|
|
|
| |
When we refactored our decision process to fix #239, we broke a subtle
corner case where extensions that are not off-by-default are to be
reported.
This further refactors that logic and adds specific tests around it to
ensure it works as expected and doesn't regress.
Closes #257
|
| |
|
|
|
|
|
|
| |
This makes the regular expression a bit more complex, and potentially
slower, but it will fix the issue where users had noqa comments with
colons followed by explanations.
Closes #178
|
| |
|
|
|
|
|
|
|
|
|
| |
When attempting to centralize all inline NoQA handling in the StyleGuide
we inadvertently broke plugins relying on it in combination with checker
state. For example, the check for E402 relies both on NoQA and the state
to determine if it has seen a non-import line. Placing NoQA on the sole
line that is not an import is more elegant than placing it on each of
the following import lines.
Closes #186
|
| |
|
|
| |
McCabe reports C90* not C* and Flake8 2 used to exclude .eggs and *.egg
|
|
|
This is an emerging best practice and there is little reason to not
follow it
|
