Resolve cli.log.warn warnings (#24551)
				
					
				
			This commit is contained in:
		
							parent
							
								
									296e4539f5
								
							
						
					
					
						commit
						767dfbbd3f
					
				| @ -102,10 +102,10 @@ def userspace_tests(qmk_firmware): | |||||||
|             qmk_userspace_validate(path) |             qmk_userspace_validate(path) | ||||||
|             cli.log.info(f'Testing userspace candidate: {{fg_cyan}}{path}{{fg_reset}} -- {{fg_green}}Valid `qmk.json`') |             cli.log.info(f'Testing userspace candidate: {{fg_cyan}}{path}{{fg_reset}} -- {{fg_green}}Valid `qmk.json`') | ||||||
|         except FileNotFoundError: |         except FileNotFoundError: | ||||||
|             cli.log.warn(f'Testing userspace candidate: {{fg_cyan}}{path}{{fg_reset}} -- {{fg_red}}Missing `qmk.json`') |             cli.log.warning(f'Testing userspace candidate: {{fg_cyan}}{path}{{fg_reset}} -- {{fg_red}}Missing `qmk.json`') | ||||||
|         except UserspaceValidationError as err: |         except UserspaceValidationError as err: | ||||||
|             cli.log.warn(f'Testing userspace candidate: {{fg_cyan}}{path}{{fg_reset}} -- {{fg_red}}Invalid `qmk.json`') |             cli.log.warning(f'Testing userspace candidate: {{fg_cyan}}{path}{{fg_reset}} -- {{fg_red}}Invalid `qmk.json`') | ||||||
|             cli.log.warn(f' -- {{fg_cyan}}{path}/qmk.json{{fg_reset}} validation error: {err}') |             cli.log.warning(f' -- {{fg_cyan}}{path}/qmk.json{{fg_reset}} validation error: {err}') | ||||||
| 
 | 
 | ||||||
|     if QMK_USERSPACE is not None: |     if QMK_USERSPACE is not None: | ||||||
|         cli.log.info(f'QMK userspace: {{fg_cyan}}{QMK_USERSPACE}') |         cli.log.info(f'QMK userspace: {{fg_cyan}}{QMK_USERSPACE}') | ||||||
|  | |||||||
| @ -15,7 +15,7 @@ def import_kbfirmware(cli): | |||||||
|     cli.log.info(f'{{style_bright}}Importing {filename.name}.{{style_normal}}') |     cli.log.info(f'{{style_bright}}Importing {filename.name}.{{style_normal}}') | ||||||
|     cli.echo('') |     cli.echo('') | ||||||
| 
 | 
 | ||||||
|     cli.log.warn("Support here is basic - Consider using 'qmk new-keyboard' instead") |     cli.log.warning("Support here is basic - Consider using 'qmk new-keyboard' instead") | ||||||
| 
 | 
 | ||||||
|     kb_name = _import_kbfirmware(data) |     kb_name = _import_kbfirmware(data) | ||||||
| 
 | 
 | ||||||
|  | |||||||
| @ -66,4 +66,4 @@ def userspace_list(cli): | |||||||
|         if is_all_keyboards(keyboard) or is_keymap_target(keyboard_folder(keyboard), keymap): |         if is_all_keyboards(keyboard) or is_keymap_target(keyboard_folder(keyboard), keymap): | ||||||
|             cli.log.info(f'Keyboard: {{fg_cyan}}{keyboard}{{fg_reset}}, keymap: {{fg_cyan}}{keymap}{{fg_reset}}{extra_args_str}') |             cli.log.info(f'Keyboard: {{fg_cyan}}{keyboard}{{fg_reset}}, keymap: {{fg_cyan}}{keymap}{{fg_reset}}{extra_args_str}') | ||||||
|         else: |         else: | ||||||
|             cli.log.warn(f'Keyboard: {{fg_cyan}}{keyboard}{{fg_reset}}, keymap: {{fg_cyan}}{keymap}{{fg_reset}}{extra_args_str} -- not found!') |             cli.log.warning(f'Keyboard: {{fg_cyan}}{keyboard}{{fg_reset}}, keymap: {{fg_cyan}}{keymap}{{fg_reset}}{extra_args_str} -- not found!') | ||||||
|  | |||||||
| @ -26,7 +26,7 @@ def git_get_version(repo_dir='.', check_dir='.'): | |||||||
|             return git_describe.stdout.strip() |             return git_describe.stdout.strip() | ||||||
| 
 | 
 | ||||||
|         else: |         else: | ||||||
|             cli.log.warn(f'"{" ".join(git_describe_cmd)}" returned error code {git_describe.returncode}') |             cli.log.warning(f'"{" ".join(git_describe_cmd)}" returned error code {git_describe.returncode}') | ||||||
|             print(git_describe.stderr) |             print(git_describe.stderr) | ||||||
|             return None |             return None | ||||||
| 
 | 
 | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user