Home GnuPG
Diffusion Arcanist 4f1141d0c59b

Improve error handling in `arc install-certificate`
4f1141d0c59bUnpublished
Actions

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Improve error handling in arc install-certificate

Summary: Fixes T9858. Reasonable typos and misunderstandings currently produce very confusing error messages.

Test Plan:

$ arc install certificate
Usage Exception: Server URI "certificate" must include a protocol and domain. It should be in the form "https://phabricator.example.com/".
  • Also used a good URI.
  • Also used no URI.

Reviewers: joshuaspence, chad

Reviewed By: chad

Maniphest Tasks: T9858

Differential Revision: https://secure.phabricator.com/D14577

Details

Provenance
epriestley <git@epriestley.com>Authored on Nov 27 2015, 5:35 PM
Parents
rARCeeaa176cfc5a: Add a linter rule to ensure that `namespace` is the first statement in a file
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rARC4f1141d0c59b: Improve error handling in `arc install-certificate` (authored by epriestley <git@epriestley.com>).Nov 27 2015, 6:05 PM

Changes (1)

PathSize
src/
workflow/

rARC4f1141d0c59b

View Options

src/workflow/ArcanistInstallCertificateWorkflow.php

Loading...