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/orc.git
The following commit(s) were added to refs/heads/main by this push:
new 2d2611ce0 ORC-1975: Improve `merge_orc_pr.py` to accept PR numbers as
a CLI argument
2d2611ce0 is described below
commit 2d2611ce09a877be0d47127f034026bda54e818b
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Fri Aug 15 14:23:45 2025 -0700
ORC-1975: Improve `merge_orc_pr.py` to accept PR numbers as a CLI argument
### What changes were proposed in this pull request?
This PR aims to improve `merge_orc_pr.py` to accept PR numbers as a CLI
argument.
### Why are the changes needed?
To provide a new easy way for the committers to use this script.
### How was this patch tested?
Manual review.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #2359 from dongjoon-hyun/ORC-1975.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
dev/merge_orc_pr.py | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/dev/merge_orc_pr.py b/dev/merge_orc_pr.py
index 12062c166..caa78fe6f 100755
--- a/dev/merge_orc_pr.py
+++ b/dev/merge_orc_pr.py
@@ -449,7 +449,11 @@ def main():
branch_names = sorted(branch_names, reverse=True)
branch_iter = iter(branch_names)
- pr_num = input("Which pull request would you like to merge? (e.g. 34): ")
+ if len(sys.argv) == 1:
+ pr_num = 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))
