Closed Bug 1601023 Opened 6 years ago Closed 3 years ago

Handle "install-certificate" inline with original command

Categories

(Conduit :: moz-phab, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mhentges, Assigned: grammar)

Details

(Keywords: conduit-triaged)

Attachments

(1 file)

Right now, if I attempt to moz-phab a patch but I don't have an API key yet, moz-phab will throw an error and tell me to run moz-phab install-certificate.

However, perhaps we should automatically run the install-certificate workflow, then retry pushing the commit. There's already precedent for this, since if your moz-phab is out-of-date, it automatically updates then performs the command (instead of erroring and telling you to moz-phab self-update)

We're following the Arcanist workflow.
I remember having this conversation with glob, but not the actual reason we've decided to not do it automatically.

Flags: needinfo?(glob)

I can't think of any reason why we couldn't do it automatically.

Flags: needinfo?(glob)
Keywords: conduit-triaged
Priority: -- → P3
Assignee: nobody → grobertson
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: