Bulk reviewing with embark

[oliverpauffley]

First off thanks for the great package!

I can successfully list PRs where I have been requested to review and using embark I can export the list. Is there a way to bulk approve PRs?

(This might not be the best idea but at work we have a 1000s of repos and often bots will raise PRs that are the same across all repos)

Thanks

[armindarvish]

@oliverpauffley As you noted, this is likely not the best idea, but if you must here are some ideas you can try:

1- Use embark all (or embark select to select multiple ones) and the call the embark action consult-gh-embark-merge-pr (by default bound to e m when you call embark-act on a pr candidate.
This is likely not the best approach though, because by default the consult-gh-embark-merge-pr loads the pr in a buffer and then runs the internal command consult-gh-pr--merge-merge so if this is slow you can try number 2 below.

2- define a new embark action to merge without loading PR:

(defun consult-gh-embark-merge-pr-noload (cand)
  "Merge the pull request in CAND without loading PR."
  (when (stringp cand)
    (consult-gh-with-host
     (consult-gh--auth-account-host)
     (funcall #'consult-gh-pr--merge-merge cand))))

and bind that to your favorite key (e.g. M) in consult-gh-embark-prs-edit-menu-map:

(keymap-set consult-gh-embark-prs-edit-menu-map "M" '("merge pr without loading" . consult-gh-embark-merge-pr-noload))

Then use the action you just defined with embark all or embark select as mentioned in 1 above.

Keep in mind that using embark-all or embark-select will run a loop with each candidate, so if you really have 1000 PRs to approve, this may still be very slow because #'consult-gh-pr--merge-merge checks the status of the PR to make sure it is mergable and interactively asks for what to do. If you want to ignore all that, then you can define consult-gh-embark-merge-pr-noload as follows:

(defun consult-gh-embark-merge-pr-noload (cand)
  "Merge the pull request in CAND without loading PR."
  (when (stringp cand)
    (consult-gh-with-host
     (consult-gh--auth-account-host)
     (funcall #'consult-gh-pr--merge-submit cand :merge t))))

This should show error messages for the PRs that cannot be merged, but I am not sure if it then continues with the rest of the selected PRs or aborts the rest of the bulk merge as well!

Obviously, I have not tested any of these (because merging PRs without reviewing them is probably not a great idea) so let me know how it goes if you end up testing them.