Update TMTC and linter script #15

Merged
meierj merged 5 commits from mueller/master into develop 2021-08-09 14:19:44 +02:00
5 changed files with 32 additions and 3 deletions

View File

@ -1,5 +1,25 @@
# TMTC Commander EIVE # 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 # Set up virtual environment
## Linux ## Linux

13
lint.py
View File

@ -4,11 +4,20 @@ import sys
def main(): 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: if os.WEXITSTATUS(status) != 0:
print("Flake8 linter errors") print("Flake8 linter errors")
sys.exit(0) 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__": if __name__ == "__main__":

0
tmtc_client_cli.py Normal file → Executable file
View File

0
tmtc_client_gui.py Normal file → Executable file
View File

@ -1 +1 @@
Subproject commit 2058b28fa167fa1cc3eb7dd69b18b4968e86ddb1 Subproject commit 94978512deb6c91a6a1456ae3a1182bf679f063a