summaryrefslogtreecommitdiff
path: root/lib/python/qmk/cli
diff options
context:
space:
mode:
authorZach White <skullydazed@gmail.com>2020-12-18 16:42:30 -0800
committerGitHub <noreply@github.com>2020-12-18 16:42:30 -0800
commitbded5f473c887fb36a2ca9d473635c1b84d0a04b (patch)
tree106cf4499145ccdbd434d5c1fbf8403f8fd9338f /lib/python/qmk/cli
parent6890090fbb929e0d1fe070b590dad2ee8a8eaa66 (diff)
simplify qmk doctor to make room for #11208 (#11242)
Diffstat (limited to 'lib/python/qmk/cli')
-rwxr-xr-xlib/python/qmk/cli/doctor.py30
1 files changed, 17 insertions, 13 deletions
diff --git a/lib/python/qmk/cli/doctor.py b/lib/python/qmk/cli/doctor.py
index 4fe318b636..25e2e239cb 100755
--- a/lib/python/qmk/cli/doctor.py
+++ b/lib/python/qmk/cli/doctor.py
@@ -281,6 +281,22 @@ def is_executable(command):
return False
+def os_tests():
+ """Determine our OS and run platform specific tests
+ """
+ platform_id = platform.platform().lower()
+
+ if 'darwin' in platform_id or 'macos' in platform_id:
+ return os_test_macos()
+ elif 'linux' in platform_id:
+ return os_test_linux()
+ elif 'windows' in platform_id:
+ return os_test_windows()
+ else:
+ cli.log.warning('Unsupported OS detected: %s', platform_id)
+ return CheckStatus.WARNING
+
+
def os_test_linux():
"""Run the Linux specific tests.
"""
@@ -317,20 +333,8 @@ def doctor(cli):
* [ ] Compile a trivial program with each compiler
"""
cli.log.info('QMK Doctor is checking your environment.')
- status = CheckStatus.OK
-
- # Determine our OS and run platform specific tests
- platform_id = platform.platform().lower()
- if 'darwin' in platform_id or 'macos' in platform_id:
- status = os_test_macos()
- elif 'linux' in platform_id:
- status = os_test_linux()
- elif 'windows' in platform_id:
- status = os_test_windows()
- else:
- cli.log.warning('Unsupported OS detected: %s', platform_id)
- status = CheckStatus.WARNING
+ status = os_tests()
cli.log.info('QMK home: {fg_cyan}%s', QMK_FIRMWARE)