github-merge-pr: fix loading .config if symbolic link is used

[maintainer-tools.git] / github-merge-pr.sh

diff --git a/github-merge-pr.sh b/github-merge-pr.sh
index 4a61a3f04381c7e3a35006633b3e587b9dd39ac8..dd08a89019cf78f41c9dc0a9a81fd45783a0b007 100755 (executable)
--- a/github-merge-pr.sh
+++ b/github-merge-pr.sh
@@ -12,7 +12,10 @@
 # REPO_NAME is optional and defaults to openwrt. It does
 # describe the repository name to use to pull PR info from.
 
+# Script .config variables
 # Github repository owner or organization name.
+# By default set to openwrt, set this line in .config to overwrite
+# the default value.
 GITHUB_REPO_OWNER="openwrt"
 
 # Your repository token, generate this token at your profile page:
@@ -20,10 +23,17 @@ GITHUB_REPO_OWNER="openwrt"
 # - Click on "Generate new token"
 # - Enter a description, e.g. "pr.sh" and pick the "repo" scope
 # - Hit "Generate token"
-#TOKEN="d41d8cd98f00b204e9800998ecf8427e"
+# Set this line in .config to provide a GITHUB_TOKEN and add comments
+#GITHUB_TOKEN="d41d8cd98f00b204e9800998ecf8427e"
 #
-# Uncomment this line to use SSH key to rebase PR branch
-# USE_SSH=1
+# Set this line in .config to use SSH key to rebase PR branch
+# GITHUB_USE_SSH=1
+
+SCRIPT_DIR="$(dirname $0)"
+[ -h $0 ] && SCRIPT_DIR="$(dirname $(readlink $0))"
+
+# Everything in .config will overwrite the default values set up
+[ -f "$SCRIPT_DIR/.config" ] && source "$SCRIPT_DIR"/.config
 
 PRID="$1"
 BRANCH="${2:-main}"
@@ -93,7 +103,7 @@ fi
 PR_USER="$(echo "$PR_INFO" | jq -r ".head.user.login")"
 PR_BRANCH="$(echo "$PR_INFO" | jq -r ".head.ref")"
 LOCAL_PR_BRANCH="$PR_BRANCH"-"$PR_USER"
-if [ "$USE_SSH" = "1" ]; then
+if [ "$GITHUB_USE_SSH" = "1" ]; then
        PR_REPO="$(echo "$PR_INFO" | jq -r ".head.repo.ssh_url")"
 else
        PR_REPO="$(echo "$PR_INFO" | jq -r ".head.repo.html_url")"
@@ -130,7 +140,7 @@ if [ "$(echo "$PR_INFO" | jq -r ".maintainer_can_modify")" == "true" ]; then
 
        # Remove any previous SoB tag if the Pull Request Author and Committer match
        if ! $GIT filter-repo --message-callback "
-               return message.replace(b\"Signed-off-by: $(git config user.name) <$(git config user.email)>\",b\"\")
+               return message.replace(b\"Signed-off-by: $(git config user.name) <$(git config user.email)>\n\",b\"\")
        " --refs $BRANCH..$LOCAL_PR_BRANCH --force; then
                echo "Failed to remove previous Committer SoB tag" >&2
                exit 9
@@ -171,7 +181,7 @@ else
                # Default close comment
                COMMENT="Thanks! Rebased on top of $BRANCH and merged!"
 
-               if [ -n "$TOKEN" ] && [ -z "$DRY_RUN" ]; then
+               if [ -n "$GITHUB_TOKEN" ] && [ -z "$DRY_RUN" ]; then
                        echo ""
                        echo "Enter a comment and hit <enter> to close the PR at Github automatically now."
                        echo "Hit <ctrl>-<c> to exit."
@@ -189,8 +199,8 @@ else
                        comment="${comment//\"/\\\"}"
                        comment="$(printf '{"body":"%s"}' "$comment")"
 
-                       if ! curl -s -o /dev/null -w "%{http_code} %{url_effective}\\n" --user "$TOKEN:x-oauth-basic" --request POST --data "$comment" "https://api.github.com/repos/$REPO/issues/$PRID/comments" || \
-                       ! curl -s -o /dev/null -w "%{http_code} %{url_effective}\\n" --user "$TOKEN:x-oauth-basic" --request PATCH --data '{"state":"closed"}' "https://api.github.com/repos/$REPO/pulls/$PRID"
+                       if ! curl -s -o /dev/null -w "%{http_code} %{url_effective}\\n" --user "$GITHUB_TOKEN:x-oauth-basic" --request POST --data "$comment" "https://api.github.com/repos/$REPO/issues/$PRID/comments" || \
+                       ! curl -s -o /dev/null -w "%{http_code} %{url_effective}\\n" --user "$GITHUB_TOKEN:x-oauth-basic" --request PATCH --data '{"state":"closed"}' "https://api.github.com/repos/$REPO/pulls/$PRID"
                        then
                                echo ""                                                     >&2
                                echo "Something failed while sending comment to the PR via ">&2