Split out "prepare-rollout" and "land-rollout" from "rollout" and make --complete-rollout default --complete-rollout was a hack back when I didn't trust rollout. It's still possible for it to go wrong, but unlikely. Especially with our diff-confirmation step, we should remove the --complete-rollout flag. It's sometimes useful to be able to prepare rollouts and land them after further editing. prepare-rollout would be useful. Which woudl effectively be the current "rollout" without --complete-rollout passed.
I think this patch is super easy. You should just do it.
Haven't written "land-rollout" yet (because it requires some fancy reason parsing from the ChangeLog), but the rest is done. Will upload.
Created attachment 47408 [details] Patch
Comment on attachment 47408 [details] Patch Excellent! I think you can get rid of MetaStep now, btw.
I expect the cq+ will fail, but if it doesn't it saves me time landing! :)
Comment on attachment 47408 [details] Patch Clearing flags on attachment: 47408 Committed r55034: <http://trac.webkit.org/changeset/55034>
All reviewed patches have been landed. Closing bug.