* Align our subprocess usage with current best practices. * remove unused import * Apply suggestions from code review Co-authored-by: Ryan <fauxpark@gmail.com> * fix the cpp invocation for older python * allow for unprompted installation * make sure qmk new-keyboard works on windows Co-authored-by: Ryan <fauxpark@gmail.com>
		
			
				
	
	
		
			27 lines
		
	
	
		
			1014 B
		
	
	
	
		
			Python
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			27 lines
		
	
	
		
			1014 B
		
	
	
	
		
			Python
		
	
	
		
			Executable File
		
	
	
	
	
| """Format python code according to QMK's style.
 | |
| """
 | |
| from subprocess import CalledProcessError, DEVNULL
 | |
| 
 | |
| from milc import cli
 | |
| 
 | |
| 
 | |
| @cli.argument('-n', '--dry-run', arg_only=True, action='store_true', help="Flag only, don't automatically format.")
 | |
| @cli.subcommand("Format python code according to QMK's style.", hidden=False if cli.config.user.developer else True)
 | |
| def pyformat(cli):
 | |
|     """Format python code according to QMK's style.
 | |
|     """
 | |
|     edit = '--diff' if cli.args.dry_run else '--in-place'
 | |
|     yapf_cmd = ['yapf', '-vv', '--recursive', edit, 'bin/qmk', 'lib/python']
 | |
|     try:
 | |
|         cli.run(yapf_cmd, check=True, capture_output=False, stdin=DEVNULL)
 | |
|         cli.log.info('Python code in `bin/qmk` and `lib/python` is correctly formatted.')
 | |
|         return True
 | |
| 
 | |
|     except CalledProcessError:
 | |
|         if cli.args.dry_run:
 | |
|             cli.log.error('Python code in `bin/qmk` and `lib/python` incorrectly formatted!')
 | |
|         else:
 | |
|             cli.log.error('Error formatting python code!')
 | |
| 
 | |
|     return False
 |