[PATCH] D142726: [Workflow] Add Release Repo sync script

[Konrad Wilhelm Kleine via Phabricator via llvm-commits]

kwk added inline comments.


================
Comment at: llvm/utils/git/sync-release-repo.sh:20-26
+MAIN_URL="https://github.com/llvm/llvm-project"
+RELEASE_URL="https://github.com/llvm/llvm-project-release-prs"
+MAIN_REMOTE=$(uuidgen)
+RELEASE_REMOTE=$(uuidgen)
+
+git remote add $MAIN_REMOTE $MAIN_URL
+git remote add $RELEASE_REMOTE $RELEASE_URL
----------------
You're only using the remote URLs once so no need for the variable.

I've re-added the CURRENT_BRANCH in order for the cleanup to go back where you came from.


================
Comment at: llvm/utils/git/sync-release-repo.sh:43
+  echo "Changes in the release remote - pushing to main remote"
+  git push origin $MERGE_BRANCH:$RELEASE_BRANCH
+fi
----------------



================
Comment at: llvm/utils/git/sync-release-repo.sh:55-56
+if ! git diff-index --quiet $RELEASE_REMOTE/$RELEASE_BRANCH; then
+  echo "Changes in upstream - pushing to origin"
+  git push upstream $MERGE_BRANCH:$RELEASE_BRANCH
+fi
----------------



================
Comment at: llvm/utils/git/sync-release-repo.sh:64
+
+  git switch main
+  git branch -D $MERGE_BRANCH
----------------
One not necessarily was on the main branch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142726/new/

https://reviews.llvm.org/D142726