2021-09-24 06:02:20 +00:00
|
|
|
'''
|
|
|
|
lint_argparse_returns
|
|
|
|
=====================
|
|
|
|
'''
|
|
|
|
import ast
|
|
|
|
import sys
|
|
|
|
|
|
|
|
from voussoirkit import pathclass
|
|
|
|
from voussoirkit import vlogging
|
|
|
|
|
|
|
|
log = vlogging.getLogger(__name__, 'lint_argparse_returns')
|
|
|
|
|
|
|
|
@vlogging.main_decorator
|
|
|
|
def main(argv):
|
|
|
|
return_status = 0
|
|
|
|
|
2021-10-03 06:40:29 +00:00
|
|
|
patterns = argv[:]
|
|
|
|
if patterns:
|
|
|
|
files = pathclass.glob_many(patterns, files=True)
|
|
|
|
else:
|
|
|
|
files = pathclass.glob('*.py', files=True)
|
|
|
|
|
2021-09-24 06:02:20 +00:00
|
|
|
for file in files:
|
|
|
|
no_py = file.replace_extension('').basename
|
2021-10-05 00:21:14 +00:00
|
|
|
text = file.read('r', encoding='utf-8')
|
2021-09-24 06:02:20 +00:00
|
|
|
try:
|
|
|
|
tree = ast.parse(text)
|
|
|
|
except Exception:
|
|
|
|
log.error('%s failed to parse.', file.absolute_path)
|
|
|
|
return_status = 1
|
|
|
|
continue
|
|
|
|
functions = [f for f in tree.body if isinstance(f, ast.FunctionDef)]
|
|
|
|
functions = [f for f in functions if f.name.endswith('_argparse') or f.name == 'main']
|
|
|
|
for function in functions:
|
|
|
|
returns = [node for node in ast.walk(function) if isinstance(node, ast.Return)]
|
|
|
|
|
|
|
|
if not returns:
|
|
|
|
log.warning('%s.%s has no return.', no_py, function.name)
|
|
|
|
return_status = 1
|
|
|
|
|
|
|
|
for ret in returns:
|
|
|
|
source = ast.get_source_segment(text, ret.value)
|
|
|
|
log.info('%s.%s returns %s.', no_py, function.name, source)
|
|
|
|
|
|
|
|
return return_status
|
|
|
|
|
|
|
|
if __name__ == '__main__':
|
|
|
|
raise SystemExit(main(sys.argv[1:]))
|