isledecomp/isle
1
0
Fork 0
mirror of https://github.com/isledecomp/isle.git synced 2026-01-24 08:41:16 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Rename github action, better argparse for linter

Browse Source
This commit is contained in:
disinvite 2023-12-11 22:40:38 -05:00
parent 775256a142
commit 38b44e1246
2 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
.github/workflows/order.yml vendored
View File

@ -1,9 +1,9 @@
name: Check order
name: Decomp marker linting
on: [push, pull_request]
jobs:
checkorder:
decomplint:
runs-on: ubuntu-latest
steps:

20
tools/decomplint/decomplint.py
View File

@ -33,7 +33,7 @@ def display_errors(alerts, filename):
print(f"{colorama.Fore.WHITE} {alert.line}")
def parse_args() -> dict:
def parse_args() -> argparse.Namespace:
p = argparse.ArgumentParser(
description="Syntax checking and linting for decomp annotation markers."
)
@ -57,8 +57,8 @@ def parse_args() -> dict:
help="Fail if syntax warnings are found and enforce is enabled.",
)
args = p.parse_args()
return vars(args)
(args, _) = p.parse_known_args()
return args
def process_files(files, module=None):
@ -85,19 +85,19 @@ def process_files(files, module=None):
def main():
args = parse_args()
if os.path.isdir(args["target"]):
files_to_check = list(walk_source_dir(args["target"]))
elif os.path.isfile(args["target"]) and is_file_cpp(args["target"]):
files_to_check = [args["target"]]
if os.path.isdir(args.target):
files_to_check = list(walk_source_dir(args.target))
elif os.path.isfile(args.target) and is_file_cpp(args.target):
files_to_check = [args.target]
else:
sys.exit("Invalid target")
(warning_count, error_count) = process_files(files_to_check, module=args["module"])
(warning_count, error_count) = process_files(files_to_check, module=args.module)
print(colorama.Style.RESET_ALL, end="")
would_fail = error_count > 0 or (warning_count > 0 and args["warnfail"])
if args["enforce"] and would_fail:
would_fail = error_count > 0 or (warning_count > 0 and args.warnfail)
if args.enforce and would_fail:
sys.exit(1)