Page MenuHomePhabricator

Improve error messages when specifying bad set or list to bin/config
ClosedPublic

Authored by epriestley on Mar 2 2015, 3:00 PM.
Tags
None
Referenced Files
F19102013: D11925.diff
Fri, Dec 5, 5:23 AM
F18855453: D11925.diff
Nov 1 2025, 5:18 AM
F18839282: D11925.id.diff
Oct 27 2025, 3:25 PM
F18820011: D11925.id28728.diff
Oct 22 2025, 11:49 AM
F18818901: D11925.diff
Oct 22 2025, 3:07 AM
F18631121: D11925.id.diff
Sep 16 2025, 2:27 PM
F18586813: D11925.diff
Sep 11 2025, 7:48 PM
F18505013: D11925.id.diff
Sep 5 2025, 12:20 AM
Subscribers

Details

Summary

Fixes T7308. Multiple users have encountered confusion around how they should specify a set or list in JSON; provide examples.

Test Plan
epriestley@orbital ~/dev/phabricator $ ./bin/config set files.image-mime-types true
Usage Exception: Config key 'files.image-mime-types' is of type 'set'. Specify it in JSON. For example:

    ./bin/config set '{"value1": true, "value2": true}'

epriestley@orbital ~/dev/phabricator $ ./bin/config set cluster.addresses true
Usage Exception: Config key 'cluster.addresses' is of type 'list<string>'. Specify it in JSON. For example:

    ./bin/config set '["a", "b", "c"]'

epriestley@orbital ~/dev/phabricator $

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve error messages when specifying bad set or list to bin/config.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 2 2015, 3:04 PM
This revision was automatically updated to reflect the committed changes.