Merge pull request #339 from epage/action

fix: Don't require sudo

action.yml

@@ -19,6 +19,7 @@ runs:
       run: $GITHUB_ACTION_PATH/action/entrypoint.sh
       shell: bash
       env:
+        INSTALL_DIR: .
         INPUT_ARGS: ${{ inputs.args }}
         INPUT_COMMITS: ${{ inputs.commits }}
 

action/entrypoint.sh

@@ -6,18 +6,25 @@ log() {
     echo -e "$1" >&2
 }
 
+_DEFAULT_INSTALL_DIR=${HOME}/bin
+_INSTALL_DIR=${INSTALL_DIR:-${_DEFAULT_INSTALL_DIR}}
 CMD_NAME="committed"
+COMMAND="${_INSTALL_DIR}/${CMD_NAME}"
 
-if [[ -z $(which ${CMD_NAME} 2>/dev/null) ]]; then
+if [[ -z ${INPUT_COMMITS:-} ]]; then
+    log "'INPUT_COMMITS' is required"
+    exit 1
+fi
+
+if [[ ! -x ${COMMAND} ]]; then
     VERSION=1.0.17
     log "Downloading 'committed' v${VERSION}"
-    wget https://github.com/crate-ci/committed/releases/download/v${VERSION}/committed-v${VERSION}-x86_64-unknown-linux-musl.tar.gz
-    sudo tar -xzvf committed-v${VERSION}-x86_64-unknown-linux-musl.tar.gz -C /usr/local/bin ./committed
+    wget --progress=dot:mega https://github.com/crate-ci/committed/releases/download/v${VERSION}/committed-v${VERSION}-x86_64-unknown-linux-musl.tar.gz
+    mkdir -p ${_INSTALL_DIR}
+    tar -xzvf committed-v${VERSION}-x86_64-unknown-linux-musl.tar.gz -C ${_INSTALL_DIR} ./${CMD_NAME}
     rm committed-v${VERSION}-x86_64-unknown-linux-musl.tar.gz
 fi
 
-COMMAND="${CMD_NAME}"
-
 echo "Linting commits:"
 git config --global --add safe.directory "$PWD"
 git log --color=always --graph --oneline "${INPUT_COMMITS}" || true
@@ -28,4 +35,4 @@ echo ""
 echo "If this fails, don't sweat it. We're trying to encourage clear communication and not hinder contributions."
 echo "If it is a reasonable issue and you lack time or feel uncomfortable fixing it yourself,"
 echo "let us know and we can mentor or fix it."
-${COMMAND} --color=always $INPUT_ARGS "${INPUT_COMMITS}"
+${COMMAND} --color=always ${INPUT_ARGS:-} "${INPUT_COMMITS}"