diff --git a/README.md b/README.md index c52a047..d0d50c4 100644 --- a/README.md +++ b/README.md @@ -1,5 +1,25 @@ # TMTC Commander EIVE +# Run Linter + +Can be used to quickly check validity of script. Install `flake8` first + +```sh +python3 -m pip install flake8 +``` + +or on Windows + +```sh +py -m pip install flake8 +``` + +and then run the `lint.py` script + +```sh +./lint.py +``` + # Set up virtual environment ## Linux diff --git a/lint.py b/lint.py index f0f66f0..4a42485 100755 --- a/lint.py +++ b/lint.py @@ -4,11 +4,20 @@ import sys def main(): - status = os.system("flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics") + # Ignore folder created by venv + exclude_dirs_flag = '--exclude bin,lib' + additional_flags_both_steps = '--count --statistics' + additional_flags_first_step = '--select=E9,F63,F7,F82 --show-source' + flake8_first_step_cmd = \ + f'flake8 . {additional_flags_both_steps} {additional_flags_first_step} {exclude_dirs_flag}' + status = os.system(flake8_first_step_cmd) if os.WEXITSTATUS(status) != 0: print("Flake8 linter errors") sys.exit(0) - os.system("flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics") + flake8_second_step_cmd = \ + f'flake8 . {additional_flags_both_steps} --exit-zero --max-complexity=10 ' \ + f'--max-line-length=127 {exclude_dirs_flag}' + os.system(flake8_second_step_cmd) if __name__ == "__main__": diff --git a/tmtc_client_cli.py b/tmtc_client_cli.py old mode 100644 new mode 100755 diff --git a/tmtc_client_gui.py b/tmtc_client_gui.py old mode 100644 new mode 100755 diff --git a/tmtccmd b/tmtccmd index 2058b28..9497851 160000 --- a/tmtccmd +++ b/tmtccmd @@ -1 +1 @@ -Subproject commit 2058b28fa167fa1cc3eb7dd69b18b4968e86ddb1 +Subproject commit 94978512deb6c91a6a1456ae3a1182bf679f063a