Add question before pushing a patch with bulk_review.sh

When running this bulk command it is always good to double check what
we are pushing to gerrit. This patch shows the commit message with
changed files list with 'git show --stat' command and then waits for an
Enter. This also makes it more visible if a git push has failed.

Note: when aborting the script the branches will remain in our local
git repository and that needs to be cleaned if we want to run the
script again. Though this is also valid if we don't cancel the script.

Change-Id: If23202be15493d2ac9bccd3e2d1af04395b18a23
This commit is contained in:
Előd Illés 2022-02-28 20:16:41 +01:00
parent 3d2301adbd
commit 4f103be7f4
1 changed files with 4 additions and 0 deletions

View File

@ -112,6 +112,10 @@ for team in "${!files_by_team_release[@]}" ; do
branch_name=${team/ /_}-${topic}
git checkout $branch_name
git show --stat
echo
echo 'Push? (Ctrl-C to cancel)'
read
git review -y -t $topic $review_args
done
git checkout master