diff options
author | skullY <skullydazed@gmail.com> | 2019-11-23 10:42:39 -0800 |
---|---|---|
committer | skullydazed <skullydazed@users.noreply.github.com> | 2019-12-01 13:40:14 -0800 |
commit | 0ed492978aea83205b73127472c3667c601d515b (patch) | |
tree | 181720a026d1d1e293845469e135e4b416baae67 | |
parent | a2c6257942275eb874f79527e1ff13410bc42fb6 (diff) |
Add pep8-naming to our python linting
-rwxr-xr-x | bin/qmk | 5 | ||||
-rwxr-xr-x | lib/python/qmk/cli/doctor.py | 2 | ||||
-rw-r--r-- | lib/python/qmk/tests/test_qmk_errors.py | 2 | ||||
-rw-r--r-- | requirements.txt | 1 |
4 files changed, 8 insertions, 2 deletions
@@ -25,6 +25,11 @@ with open(os.path.join(qmk_dir, 'requirements.txt'), 'r') as fd: line = line.split('#')[0] module = line.split('=')[0] if '=' in line else line + + if module in ['pep8-naming']: + # Not every module is importable by its own name. + continue + if not find_spec(module): print('Could not find module %s!' % module) print('Please run `pip3 install -r requirements.txt` to install the python dependencies.') diff --git a/lib/python/qmk/cli/doctor.py b/lib/python/qmk/cli/doctor.py index 1010eafb33..2421e520e0 100755 --- a/lib/python/qmk/cli/doctor.py +++ b/lib/python/qmk/cli/doctor.py @@ -39,7 +39,7 @@ def doctor(cli): ok = False # Determine our OS and run platform specific tests - OS = platform.system() + OS = platform.system() # noqa (N806), uppercase name is ok in this instance if OS == "Darwin": cli.log.info("Detected {fg_cyan}macOS.") diff --git a/lib/python/qmk/tests/test_qmk_errors.py b/lib/python/qmk/tests/test_qmk_errors.py index 1d8690b7ef..948e7ef741 100644 --- a/lib/python/qmk/tests/test_qmk_errors.py +++ b/lib/python/qmk/tests/test_qmk_errors.py @@ -1,7 +1,7 @@ from qmk.errors import NoSuchKeyboardError -def test_NoSuchKeyboardError(): +def test_nosuchkeyboarderror(): try: raise NoSuchKeyboardError("test message") except NoSuchKeyboardError as e: diff --git a/requirements.txt b/requirements.txt index 033b688fc6..074b11a8ce 100644 --- a/requirements.txt +++ b/requirements.txt @@ -6,3 +6,4 @@ colorama hjson nose2 flake8 +pep8-naming |