Merge pull request #1264 from edx/gprice/fix-release-email-script
Fix bug that omits commits in release email script
This commit is contained in:
@@ -36,15 +36,13 @@ echo
|
|||||||
echo '||Author||Changes||Commit Link||Testing Notes||'
|
echo '||Author||Changes||Commit Link||Testing Notes||'
|
||||||
|
|
||||||
for EMAIL in $RESPONSIBLE; do
|
for EMAIL in $RESPONSIBLE; do
|
||||||
AUTHORED_BY="$LOG_CMD --author=<${EMAIL}>"
|
AUTHORED_BY=$($LOG_CMD --author="<${EMAIL}>" --format='tformat:%h')
|
||||||
COMMITTED_BY="$LOG_CMD --committer=<${EMAIL}>"
|
COMMITTED_BY=$($LOG_CMD --committer="<${EMAIL}>" --format='tformat:%h')
|
||||||
COMMITTED_NOT_AUTHORED="$COMMITTED_BY $($AUTHORED_BY --format='tformat:^%h')"
|
ALL_COMMITS=$(for HASH in $AUTHORED_BY $COMMITTED_BY; do echo $HASH; done | sort | uniq)
|
||||||
|
|
||||||
$AUTHORED_BY --format="tformat:|$EMAIL|%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" | head -n 1
|
EMAIL_COL="$EMAIL"
|
||||||
$AUTHORED_BY --format="tformat:| |%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" | tail -n +2
|
for HASH in $ALL_COMMITS; do
|
||||||
|
git log --format="tformat:|$EMAIL_COL|%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" -n 1 $HASH
|
||||||
if [[ $($COMMITTED_NOT_AUTHORED) != "" ]]; then
|
EMAIL_COL=" "
|
||||||
$COMMITTED_NOT_AUTHORED --format="tformat:|$EMAIL|%s|[commit|https://github.com/edx/edx-platform/commit/%h]|Committed, didn't author|" | head -n 1
|
done
|
||||||
$COMMITTED_NOT_AUTHORED --format="tformat:| |%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" | tail -n +2
|
|
||||||
fi
|
|
||||||
done
|
done
|
||||||
Reference in New Issue
Block a user