Skip to content

Commit

Permalink
Merge pull request #478 from ZeusWPI/suppress-git
Browse files Browse the repository at this point in the history
Suppress git output so we get less emails
  • Loading branch information
niknetniko authored Nov 30, 2023
2 parents 0f27f85 + 41f446d commit 344ddd5
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions server/scraper/resto.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,13 @@ function usage() {
echo " push true or false, depending on if you want to push to the repo or not"
}

function suppress
{
TMP=$(mktemp)
${1+"$@"} > "$TMP" 2>&1 || ( echo "$* exited with $?" && cat "$TMP" )
rm "$TMP"
}

if [[ $# -lt 2 ]]; then
echo "error: 2 operands are required" >&2
usage
Expand Down Expand Up @@ -60,6 +67,6 @@ git diff-index --quiet HEAD || git commit -m "Scraper: new data from $today"

# Porcelain prevents git from writing non-errors to stderr, resulting in emails
if [[ "$push" == true ]]; then
git pull
git push --porcelain
suppress git pull
suppress git push --porcelain
fi

0 comments on commit 344ddd5

Please sign in to comment.