#!/usr/bin/env bash
# shellcheck disable=SC2230
#  vim:ts=4:sts=4:sw=4:et
#
#  Author: Hari Sekhon
#  Date: 2015-05-25 01:38:24 +0100 (Mon, 25 May 2015)
#
#  https://github.com/HariSekhon/DevOps-Bash-tools
#
#  License: see accompanying Hari Sekhon LICENSE file
#
#  If you're using my code you're welcome to connect with me on LinkedIn and optionally send me feedback to help improve or steer this or other code I publish
#
#  https://www.linkedin.com/in/HariSekhon
#

set -euo pipefail
[ -n "${DEBUG:-}" ] && set -x
srcdir="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"

# shellcheck disable=SC1090,SC1091
. "$srcdir/lib/utils.sh"

# shellcheck source=lib/utils.sh
. "$srcdir/lib/python.sh"

# shellcheck disable=SC2034,SC2154
usage_description="
Recurses a given directory tree or \$PWD, finding all Python and Jython files and validating them using PyLint
"

help_usage "$@"

directory="${1:-.}"
shift ||:

# maxdepth 2 to avoid recursing submodules which have their own checks
files="$(find_python_jython_files "$directory" -maxdepth 2)"

if [ -z "$files" ]; then
    # this trick allows importing or calling as script
    # shellcheck disable=SC2317
    return 0 &>/dev/null || :
    # shellcheck disable=SC2317
    exit 0
fi

section "P y L i n t"

start_time="$(start_timer)"

if [ -n "${NOPYLINT:-}" ]; then
    echo "\$NOPYLINT environment variable set, skipping PyLint error checks"
elif [ -n "${QUICK:-}" ]; then
    echo "\$QUICK environment variable set, skipping PyLint error checks"
else
    if type -P pylint &>/dev/null; then
        type -P pylint
        pylint --version
        echo
        # Can't do this in one pass because pylint -E raises wrong-import-position when it doesn't individually and refuses to respect --disable
        #prog_list="
        while read -r filename; do
            #echo "checking if $filename is excluded"
            isExcluded "$filename" && continue
            #echo "added $filename for testing"
            #prog_list="$prog_list $filename"
            echo "pylint -E $filename $*"
            pylint -E "$filename" "$@"
        done <<< "$files"
        #echo
        #echo "Checking for coding errors:"
        #echo
        #echo "pylint -E $prog_list"
        #echo
        #pylint -E $prog_list
        hr; echo
    fi
fi

time_taken "$start_time"
section2 "PyLint checks passed"
echo
