Browse Source

Fix edge-case with config

Without this check, users can lock themselves out by enabling developer
mode, than disabling the dependencies. They wouldn't be able to turn off
developer mode as none of the subcommands (including 'config') would
work.
Erovia 5 years ago
parent
commit
8fe5c718b4
1 changed files with 5 additions and 4 deletions
  1. 5 4
      bin/qmk

+ 5 - 4
bin/qmk

@@ -4,7 +4,6 @@
 import os
 import sys
 from importlib.util import find_spec
-from time import strftime
 from pathlib import Path
 
 # Add the QMK python libs to our path
@@ -52,9 +51,11 @@ import milc  # noqa
 
 # For developers additional modules are needed
 if milc.cli.config.user.developer:
-    developer = True
-    _check_modules('requirements-dev.txt')
-
+    # Do not run the check for 'config',
+    # so users can turn off developer mode
+    if len(sys.argv) == 1 or (len(sys.argv) > 1 and 'config' != sys.argv[1]):
+        developer = True
+        _check_modules('requirements-dev.txt')
 
 milc.EMOJI_LOGLEVELS['INFO'] = '{fg_blue}Ψ{style_reset_all}'