Remove is_defined() approach

This commit is contained in:
Gömöri Zsolt 2019-11-01 10:19:20 +01:00
parent 760c9be2dc
commit ddcd35eaa8

View File

@ -24,10 +24,6 @@ EOF
git config --global user.name "GitHub Actions" git config --global user.name "GitHub Actions"
} }
is_defined() {
[ ! -z "${1}" ]
}
# This section only runs if there have been file changes # This section only runs if there have been file changes
echo "Checking for uncommitted changes in the git working tree." echo "Checking for uncommitted changes in the git working tree."
if ! git diff --quiet if ! git diff --quiet
@ -39,21 +35,13 @@ then
# Switch to branch from current Workflow run # Switch to branch from current Workflow run
git checkout $INPUT_BRANCH git checkout $INPUT_BRANCH
if is_defined "${INPUT_FILE_PATTERN}"; then echo "INPUT_FILE_PATTERN: ${INPUT_FILE_PATTERN}"
echo "INPUT_FILE_PATTERN: ${INPUT_FILE_PATTERN}"
git add "${INPUT_FILE_PATTERN}" git add "${INPUT_FILE_PATTERN}"
else
git add .
fi
if is_defined "${INPUT_COMMIT_OPTIONS}"; then echo "INPUT_COMMIT_OPTIONS: ${INPUT_COMMIT_OPTIONS}"
echo "INPUT_COMMIT_OPTIONS: ${INPUT_COMMIT_OPTIONS}"
git commit -m "$INPUT_COMMIT_MESSAGE" --author="$GITHUB_ACTOR <$GITHUB_ACTOR@users.noreply.github.com>" "${INPUT_COMMIT_OPTIONS}" git commit -m "$INPUT_COMMIT_MESSAGE" --author="$GITHUB_ACTOR <$GITHUB_ACTOR@users.noreply.github.com>" "${INPUT_COMMIT_OPTIONS}"
else
git commit -m "$INPUT_COMMIT_MESSAGE" --author="$GITHUB_ACTOR <$GITHUB_ACTOR@users.noreply.github.com>"
fi
git push --set-upstream origin "HEAD:$INPUT_BRANCH" git push --set-upstream origin "HEAD:$INPUT_BRANCH"
else else