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
Branch
conf1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4701
Build 4715: [Placeholder Plan] Wait for 30 Seconds

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.