zhengruifeng opened a new pull request, #56120: URL: https://github.com/apache/spark/pull/56120
### What changes were proposed in this pull request? Add a fail-hard check in `dev/merge_spark_pr.py` that aborts when the target PR is a draft (`pr["draft"] == True`). The check sits next to the existing `[WIP]`/`[DO-NOT-MERGE]` title guard. ### Why are the changes needed? A draft PR is, by definition, not ready for merge. The script today only inspects the title for `[WIP]`/`[DO-NOT-MERGE]`, so a committer who runs it against a draft PR whose title looks final can still kick off the merge flow. Checking the GitHub API's `draft` flag closes that gap. ### Does this PR introduce _any_ user-facing change? No. Committer tooling only. ### How was this patch tested? Existing doctests in `dev/merge_spark_pr.py` still pass (`python3 -m doctest dev/merge_spark_pr.py` — 57/57). ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.7) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
