summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorQMK Bot <hello@qmk.fm>2019-11-16 07:10:19 +0000
committerQMK Bot <hello@qmk.fm>2019-11-16 07:10:19 +0000
commit7891de7f6d64431cedbd580a3f7863533dc8be88 (patch)
treea4fc89d7821126c1d101cf7acea75efcecdbc7a9
parent897888db419239f013561b155de5993b1966820e (diff)
downloadqmk_firmware-7891de7f6d64431cedbd580a3f7863533dc8be88.tar.gz
qmk_firmware-7891de7f6d64431cedbd580a3f7863533dc8be88.zip
format code according to conventions [skip ci]
-rwxr-xr-xlib/python/qmk/cli/compile.py3
-rw-r--r--lib/python/qmk/cli/flash.py3
-rw-r--r--lib/python/qmk/commands.py4
-rw-r--r--lib/python/qmk/tests/test_cli_commands.py2
4 files changed, 8 insertions, 4 deletions
diff --git a/lib/python/qmk/cli/compile.py b/lib/python/qmk/cli/compile.py
index c7093d4215..5c29800096 100755
--- a/lib/python/qmk/cli/compile.py
+++ b/lib/python/qmk/cli/compile.py
@@ -38,13 +38,12 @@ def compile(cli):
# Generate the keymap
keymap_path = qmk.path.keymap(user_keymap['keyboard'])
cli.log.info('Creating {fg_cyan}%s{style_reset_all} keymap in {fg_cyan}%s', user_keymap['keymap'], keymap_path)
-
+
# Compile the keymap
command = compile_configurator_json(cli.args.filename)
cli.log.info('Wrote keymap to {fg_cyan}%s/%s/keymap.c', keymap_path, user_keymap['keymap'])
-
elif cli.config.compile.keyboard and cli.config.compile.keymap:
# Generate the make command for a specific keyboard/keymap.
command = create_make_command(cli.config.compile.keyboard, cli.config.compile.keymap)
diff --git a/lib/python/qmk/cli/flash.py b/lib/python/qmk/cli/flash.py
index 8f7bb55a22..37556aaafd 100644
--- a/lib/python/qmk/cli/flash.py
+++ b/lib/python/qmk/cli/flash.py
@@ -32,6 +32,7 @@ def print_bootloader_help():
cli.echo('\tst-link-cli')
cli.echo('For more info, visit https://docs.qmk.fm/#/flashing')
+
@cli.argument('-bl', '--bootloader', default='flash', help='The flash command, corresponding to qmk\'s make options of bootloaders.')
@cli.argument('filename', nargs='?', arg_only=True, help='The configurator export JSON to compile. Use this if you dont want to specify a keymap and keyboard.')
@cli.argument('-km', '--keymap', help='The keymap to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.')
@@ -84,4 +85,4 @@ def flash(cli):
return False
cli.log.info('Flashing keymap with {fg_cyan}%s\n\n', ' '.join(command))
- subprocess.run(command) \ No newline at end of file
+ subprocess.run(command)
diff --git a/lib/python/qmk/commands.py b/lib/python/qmk/commands.py
index 9fbf00f165..f83a89578e 100644
--- a/lib/python/qmk/commands.py
+++ b/lib/python/qmk/commands.py
@@ -3,6 +3,7 @@
import json
import qmk.keymap
+
def create_make_command(keyboard, keymap, target=None):
"""Create a make compile command
@@ -23,6 +24,7 @@ def create_make_command(keyboard, keymap, target=None):
return ['make', ':'.join((keyboard, keymap))]
return ['make', ':'.join((keyboard, keymap, target))]
+
def parse_configurator_json(configurator_filename):
"""Open and parse a configurator json export
"""
@@ -31,6 +33,7 @@ def parse_configurator_json(configurator_filename):
file.close()
return user_keymap
+
def compile_configurator_json(configurator_filename, bootloader=None):
"""Convert a configurator export JSON file into a C file
@@ -54,4 +57,3 @@ def compile_configurator_json(configurator_filename, bootloader=None):
if bootloader is None:
return create_make_command(user_keymap['keyboard'], user_keymap['keymap'])
return create_make_command(user_keymap['keyboard'], user_keymap['keymap'], bootloader)
-
diff --git a/lib/python/qmk/tests/test_cli_commands.py b/lib/python/qmk/tests/test_cli_commands.py
index dcab8bdae4..3f75cef3e1 100644
--- a/lib/python/qmk/tests/test_cli_commands.py
+++ b/lib/python/qmk/tests/test_cli_commands.py
@@ -13,10 +13,12 @@ def test_cformat():
def test_compile():
assert check_subcommand('compile', '-kb', 'handwired/onekey/pytest', '-km', 'default').returncode == 0
+
def test_flash():
assert check_subcommand('flash', '-b').returncode == 1
assert check_subcommand('flash').returncode == 1
+
def test_config():
result = check_subcommand('config')
assert result.returncode == 0