From 40bf0b549bb7f4569ea2ee25e7fa3244ba483683 Mon Sep 17 00:00:00 2001 From: jasquat Date: Wed, 16 Nov 2022 11:42:56 -0500 Subject: [PATCH] updated run_pyl to work without no-term w/ burnettk --- bin/run_pyl | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/bin/run_pyl b/bin/run_pyl index 93195bd12..fa729ebbb 100755 --- a/bin/run_pyl +++ b/bin/run_pyl @@ -12,8 +12,27 @@ python_projects=( spiffworkflow-backend ) -function run_fix_docstrings() { - fix_python_docstrings $(get_top_level_directories_containing_python_files) +function get_python_dirs() { + (git ls-tree -r HEAD --name-only | grep -E '\.py$' | awk -F '/' '{print $1}' | sort | uniq | grep -v '\.' | grep -Ev '^(bin|migrations)$') || echo '' +} + +function run_autoflake() { + if ! command -v autoflake8 >/dev/null ; then + pip install autoflake8 + asdf reshim python + fi + + if ! command -v autopep8 >/dev/null ; then + pip install autopep8 + asdf reshim python + fi + + python_dirs=$(get_python_dirs) + python_files=$(find $python_dirs -type f -name "*.py" ! -name '.null-ls*' ! -name '_null-ls*') + + autoflake8 --in-place --remove-unused-variables --remove-duplicate-keys --expand-star-imports --exit-zero-even-if-changed $python_files + autoflake --in-place --remove-all-unused-imports $python_files + autopep8 --in-place $python_files } function run_pre_commmit() { @@ -22,15 +41,15 @@ function run_pre_commmit() { for python_project in "${python_projects[@]}" ; do pushd "$python_project" - run_fix_docstrings || run_fix_docstrings + run_autoflake || run_autoflake popd done run_pre_commmit || run_pre_commmit for python_project in "${python_projects[@]}"; do pushd "$python_project" - poet i - poet mypy - poet test + poetry install + poetry run mypy $(get_python_dirs) + poetry run coverage run --parallel -m pytest popd done