Page MenuHomePhabricator

Show current config and source using arc set-config --show
ClosedPublic

Authored by iodragon on Dec 30 2013, 11:28 AM.
Tags
None
Referenced Files
F13815948: D7851.diff
Thu, Sep 19, 12:13 PM
Unknown Object (File)
Tue, Sep 10, 9:19 AM
Unknown Object (File)
Mon, Sep 9, 1:34 AM
Unknown Object (File)
Wed, Sep 4, 4:57 PM
Unknown Object (File)
Sun, Sep 1, 11:39 AM
Unknown Object (File)
Aug 14 2024, 8:23 AM
Unknown Object (File)
Aug 10 2024, 10:58 AM
Unknown Object (File)
Aug 1 2024, 8:42 PM
Subscribers

Details

Summary

arc set-config --show only show the user config
It would be better to contain local/global/system config

Test Plan

set config by local/global/system/user/project, and check the result of arc set-config --show

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Some style stuff inline, I'll clean that up in the pull.

src/workflow/ArcanistSetConfigWorkflow.php
139–144

Style stuff:

  • Indent two spaces, not 4.
  • Conventionally, use head() / head_key() over reset() + each(). They have simpler semantics.
  • Avoid redundant empty() on defined variables. if (empty($x)) is always equivalent to if (!$x) for defined $x.
  • Prefer {$variable} in strings over $variable.
  • For printf() style methods, prefer %s over {$variable} in a string. For example, if we eventually introduce a source called **stars**, it will render incorrectly (with a bold style) if embedded in the string directly.