why does approver exist instead of return?

[rjbs]

mint-tag/lib/App/MintTag.pm

Lines 65 to 69 in 4296078

	if ($self->interactive) {
	# exits on lack of user confirmation
	my $approver = App::MintTag::Approver->new($self->config);
	$approver->confirm_plan;
	}

Seems strange to me to let the approver exit, instead of just letting it signal approval. Then the caller can do the right thing. Minor, but we're usually well served to keep things like "exit" near the top level if we can.

[mmcclimon]

heh oops, mistyped an issue number in the commit message.