Commit 65a50875 authored by Aral Balkan's avatar Aral Balkan

Made the merge message better/submit optional.

parent dd878827
Pipeline #488 skipped
......@@ -36,8 +36,8 @@
showUsage(){
echo "\nUsage:\n"
echo " better/submit <trackers/sites/whitelist>/<domain> \"merge message\" : submit domain for merging\n"
echo "Submits your latest saved changes to the remote Git repository and opens a merge request with the passed merge message.\n"
echo " better/submit <trackers/sites/whitelist>/<domain> [\"merge message\"] : submit domain for merging\n"
echo "Submits your latest saved changes to the remote Git repository and opens a merge request, optionally using the passed merge message.\n"
exit 0
}
......@@ -259,7 +259,8 @@ processOtherArgumentsHook(){
# Now, let’s validate for the second argument, the message.
#
if [ "$#" -eq 0 ]; then
tooFewArgumentsError "the merge message is missing"
# No merge message. That’s OK, it’s optional.
mergeMessage=""
else
# Use whatever we’re sent as the commit message.
mergeMessage="$@"
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment