Skip to content
Snippets Groups Projects
Commit 3194d2db authored by Jean-Paul Calderone's avatar Jean-Paul Calderone
Browse files

actually use error return values instead of completely exiting

parent 70b1d137
Branches
No related tags found
2 merge requests!328update production,!318Automatically create production update MRs
......@@ -16,7 +16,10 @@ main() {
shift
# If there have been no changes we'll just abandon this update.
ensure_changes "$SOURCE_BRANCH" "$TARGET_BRANCH"
if ! ensure_changes "$SOURCE_BRANCH" "$TARGET_BRANCH"; then
echo "No changes."
exit 0
fi
local NOTES=$(describe_update "$SOURCE_BRANCH" "$TARGET_BRANCH")
......@@ -30,8 +33,7 @@ ensure_changes() {
shift
if [ $(git rev-parse "$SOURCE_BRANCH") -eq $(git rev-parse "$TARGET_BRANCH") ]; then
echo "No changes."
exit 0
return 1
fi
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment