This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-kubernetes-operator.git


The following commit(s) were added to refs/heads/main by this push:
     new fb6a5d8  [SPARK-53410] Improve `merge_spark_pr.py` to accept PR 
numbers as a CLI argument
fb6a5d8 is described below

commit fb6a5d87d01ad144a1020d8e9bc9d2152e530e77
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Wed Aug 27 22:20:51 2025 -0700

    [SPARK-53410] Improve `merge_spark_pr.py` to accept PR numbers as a CLI 
argument
    
    ### What changes were proposed in this pull request?
    
    This PR aims to improve `merge_spark_pr.py` to accept PR numbers as a CLI 
argument.
    
    ### Why are the changes needed?
    
    To provide an alternative shortcut.
    
    **BEFORE**
    
    ```
    $ dev/merge_spark_pr.py
    git rev-parse --abbrev-ref HEAD
    Which pull request would you like to merge? (e.g. 34): 302
    
================================================================================
    ```
    
    **AFTER (Additional usage)**
    
    ```
    $ dev/merge_spark_pr.py 302
    git rev-parse --abbrev-ref HEAD
    Start to merge pull request #52020
    
================================================================================
    ```
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Manual test because this is irrelevant to the CI results.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #302 from dongjoon-hyun/SPARK-53410.
    
    Authored-by: Dongjoon Hyun <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 dev/merge_spark_pr.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/dev/merge_spark_pr.py b/dev/merge_spark_pr.py
index 9a8d39f..a2a89f1 100755
--- a/dev/merge_spark_pr.py
+++ b/dev/merge_spark_pr.py
@@ -597,7 +597,11 @@ def main():
     branch_names = sorted(branch_names, reverse=True)
     branch_iter = iter(branch_names)
 
-    pr_num = bold_input("Which pull request would you like to merge? (e.g. 
34): ")
+    if len(sys.argv) == 1:
+        pr_num = bold_input("Which pull request would you like to merge? (e.g. 
34): ")
+    else:
+        pr_num = sys.argv[1]
+        print("Start to merge pull request #%s" % (pr_num))
     pr = get_json("%s/pulls/%s" % (GITHUB_API_BASE, pr_num))
     pr_events = get_json("%s/issues/%s/events" % (GITHUB_API_BASE, pr_num))
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to