Overall, the series looks pretty solid to me - which is why I've got a
handful of small nits to relay. :)
On Tue, Apr 06, 2021 at 06:47:46PM +0000, Derrick Stolee via GitGitGadget wrote:
> -static int fetch_remote(const char *remote, struct maintenance_run_opts *opts)
> +static int fetch_remote(struct remote *remote, void *cbdata)
> {
> + struct maintenance_run_opts *opts = cbdata;
[snip]
> if (opts->quiet)
I worry about the lack of null-checking here.
- Emily
