fix: changes the logic to handle dry-run with --write option and corrects the logic as it will never have changes on dry-run

This commit is contained in:
Fabricio Sander
2021-04-13 16:34:40 -03:00
committed by GitHub
parent c2e2730594
commit 634a3d2075

View File

@@ -25,6 +25,10 @@ _git_changed() {
[[ -n "$(git status -s)" ]]
}
_git_changes() {
git diff
}
# PROGRAM
echo "Installing prettier..."
case $INPUT_PRETTIER_VERSION in
@@ -56,13 +60,12 @@ prettier $INPUT_PRETTIER_OPTIONS || { PRETTIER_RESULT=$?; echo "Problem running
# To keep runtime good, just continue if something was changed
if _git_changed; then
# case when --write is used with dry-run so if something is unpretty there will always have _git_changed
if $INPUT_DRY; then
if [ "$PRETTIER_RESULT" -eq 1 ]; then
echo "Prettier found unpretty files!"
exit 1
else
echo "No unpretty files! Finishing dry-run."
fi
echo "Unpretty Files Changes:"
_git_changes
echo "Finishing dry-run. Exiting before committing."
exit 1
else
# Calling method to configure the git environemnt
_git_setup
@@ -91,5 +94,13 @@ if _git_changed; then
echo "Changes pushed successfully."
fi
else
# case when --check is used so there will never have something to commit but there are unpretty files
if [ "$PRETTIER_RESULT" -eq 1 ]; then
echo "Prettier found unpretty files!"
exit 1
else
echo "Finishing dry-run."
fi
echo "No unpretty files!"
echo "Nothing to commit. Exiting."
fi