Clean shell scripts (#398)

1 files changed, 20 insertions, 20 deletions

diff --git a/scripts/release.sh b/scripts/release.sh
index 8c440975f..6f4da6a29 100755
--- a/scripts/release.sh
+++ b/scripts/release.sh
@@ -1,11 +1,13 @@
 #!/bin/bash
 
+set -eu
+
 shutdown() {
   # Get our process group id
-  PGID=$(ps -o pgid= $$ | grep -o [0-9]*)
+  PGID=$(ps -o pgid= $$ | grep -o "[0-9]*")
 
   # Kill it in a new new process group
-  setsid kill -- -$PGID
+  setsid kill -- -"$PGID"
   exit 0
 }
 
@@ -16,7 +18,7 @@ if [ -z "$1" ]; then
   exit -1
 fi
 
-if [ -z $GITHUB_TOKEN ]; then
+if [ -z "$GITHUB_TOKEN" ]; then
   echo "Need GITHUB_TOKEN env set."
   exit -1
 fi
@@ -33,43 +35,41 @@ zip_name="peertube-$version.zip"
 
 changelog=$(awk -v version="$version" '/## v/ { printit = $2 == version }; printit;' CHANGELOG.md | grep -v "$version" | sed '1{/^$/d}')
 
-echo -e "Changelog will be:\n"
-echo "$changelog"
-echo
+printf "Changelog will be:\n%s\n" "$changelog"
 
 read -p "Are you sure to release? " -n 1 -r
 echo
 if [[ ! $REPLY =~ ^[Yy]$ ]]
 then
-  [[ "$0" = "$BASH_SOURCE" ]] && exit 0
+  exit 0
 fi
 
-cd ./client || exit -1
-npm version --no-git-tag-version --no-commit-hooks $1 || exit -1
+( cd client
+  npm version --no-git-tag-version --no-commit-hooks "$1"
+)
 
-cd ../ || exit -1
-npm version -f --no-git-tag-version --no-commit-hooks $1 || exit -1
+npm version -f --no-git-tag-version --no-commit-hooks "$1"
 
-git commit package.json client/package.json -m "Bumped to version $version" || exit -1
+git commit package.json client/package.json -m "Bumped to version $version"
 git tag -s -a "$version" -m "$version"
 
-npm run build || exit -1
-rm "./client/dist/stats.json" || exit -1
+npm run build
+rm "./client/dist/stats.json"
 
-cd ../ || exit -1
+cd ..
 
-ln -s "PeerTube" "$directory_name" || exit -1
+ln -s "PeerTube" "$directory_name"
 zip -r "PeerTube/$zip_name" "$directory_name/CREDITS.md" "$directory_name/FAQ.md" \
                             "$directory_name/LICENSE" "$directory_name/README.md" \
                             "$directory_name/client/dist/" "$directory_name/client/yarn.lock" \
                             "$directory_name/client/package.json" "$directory_name/config" \
                             "$directory_name/dist" "$directory_name/package.json" \
                             "$directory_name/scripts" "$directory_name/support" \
-                            "$directory_name/tsconfig.json" "$directory_name/yarn.lock" \
-                            || exit -1
-rm "$directory_name" || exit -1
+                            "$directory_name/tsconfig.json" "$directory_name/yarn.lock"
+
+rm "$directory_name"
 
-cd "PeerTube" || exit -1
+cd "PeerTube"
 
 git push origin --tag