[PR] [ZEPPELIN-6404] Add Go CLI tool for merging pull requests [zeppelin]

["jongyoul (via GitHub)" <gi...@apache.org>]

jongyoul opened a new pull request, #5182:
URL: https://github.com/apache/zeppelin/pull/5182

   ## Summary
   
   - Rewrite `dev/merge_zeppelin_pr.py` as a single-file Go CLI (`dev/merge-pr.go`)
   - No external dependencies — uses only Go standard library (`flag` package)
   - Runs directly with `go run dev/merge-pr.go --pr <number> [flags]` (no `go.mod` needed)
   - Non-interactive, AI-agent friendly design
   
   ## Motivation
   
   The existing Python merge script (`dev/merge_zeppelin_pr.py`) requires a Python virtual environment with dependencies (`jira`, `python-dotenv`), and is interactive (prompts for user input). This makes it difficult to use from CI/CD or AI agent workflows.
   
   The Go version:
   - Requires only a Go toolchain (no venv, no pip install)
   - All flags are CLI arguments — no interactive prompts
   - Single file, ~536 lines, zero external dependencies
   
   ## Usage
   
   ```bash
   # Dry run (shows PR info without merging)
   go run dev/merge-pr.go --pr 5167 --dry-run
   
   # Merge and resolve JIRA
   go run dev/merge-pr.go --pr 5167 --resolve-jira --fix-versions 0.13.0
   
   # Merge and cherry-pick into release branches
   go run dev/merge-pr.go --pr 5167 --release-branches branch-0.12,branch-0.11
   ```
   
   ## Flags
   
   | Flag | Description |
   |------|-------------|
   | `--pr` | Pull request number (required) |
   | `--target` | Target branch (default: PR base branch) |
   | `--fix-versions` | JIRA fix version(s), comma-separated |
   | `--release-branches` | Release branch(es) to cherry-pick into, comma-separated |
   | `--resolve-jira` | Resolve associated JIRA issue(s) |
   | `--dry-run` | Show what would be done without making changes |
   | `--push-remote` | Git remote for pushing (default: `apache`) |
   | `--github-token` | GitHub OAuth token (env: `GITHUB_OAUTH_KEY`) |
   | `--jira-token` | JIRA access token (env: `JIRA_ACCESS_TOKEN`) |
   
   ## Test Plan
   
   - [x] `go run dev/merge-pr.go --help` — shows usage
   - [x] `go run dev/merge-pr.go --pr 5167 --dry-run` — fetches PR info correctly
   - [ ] Full merge test with a real PR
   - [ ] Cherry-pick into release branch test
   - [ ] JIRA resolution test


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [ZEPPELIN-6404] Add Go CLI tool for merging pull requests [zeppelin]

["jongyoul (via GitHub)" <gi...@apache.org>]

jongyoul closed pull request #5182: [ZEPPELIN-6404] Add Go CLI tool for merging pull requests
URL: https://github.com/apache/zeppelin/pull/5182


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org