fix(action): Don't pre-filter checked files

This reverts commit fc7f517466.

This has two problems
- This doesn't correctly handle spaces, likely needing #708
- This overrides excludes, see #347

This also has a weird cost/benefit balance because this requires enough
repo history to do the analysis which takes time to pull down.

Rather than waiting until the relevant changes are in to make this work,
I'm pulling this out for now.

Fixes #806
This commit is contained in:
Ed Page 2023-08-21 12:25:34 -05:00
parent 1072809e63
commit 286609cb28

View file

@ -14,25 +14,11 @@ CMD_NAME="typos"
COMMAND="${_INSTALL_DIR}/${CMD_NAME}" COMMAND="${_INSTALL_DIR}/${CMD_NAME}"
TARGET=${INPUT_FILES:-"."} TARGET=${INPUT_FILES:-"."}
if [[ -n "${GITHUB_BASE_REF:-}" ]]; then
BASE_REF=HEAD~ # HACK: GITHUB_BASE_REF is failing the `--verify` but `HEAD~ should be the same for pull requests
git config --global --add safe.directory "$PWD"
if git rev-parse --verify ${BASE_REF} 2>/dev/null ; then
log "Limiting checks to ${GITHUB_BASE_REF}...HEAD"
TARGET=$(git diff ${BASE_REF}...HEAD --name-only --diff-filter=AM -- ${TARGET})
if [[ -z "${TARGET:-}" ]]; then
log "INPUT_FILES are unchanged"
exit 0
fi
else
log "WARN: Not limiting checks to ${BASE_REF}...HEAD, ${GITHUB_BASE_REF} is not available"
fi
fi
if [[ -z $(ls ${TARGET} 2>/dev/null) ]]; then if [[ -z $(ls ${TARGET} 2>/dev/null) ]]; then
log "ERROR: Input files (${TARGET}) not found" log "ERROR: Input files (${TARGET}) not found"
exit 1 exit 1
fi fi
if [[ ! -x ${COMMAND} ]]; then if [[ ! -x ${COMMAND} ]]; then
VERSION=1.16.7 VERSION=1.16.7
log "Downloading 'typos' v${VERSION}" log "Downloading 'typos' v${VERSION}"