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

Authored by iodragon on Dec 30 2013, 11:28 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	May 28 2025, 1:36 AM

	Unknown Object (File)
	May 27 2025, 10:38 PM

	Unknown Object (File)
	May 20 2025, 7:46 AM

	Unknown Object (File)
	Apr 25 2025, 11:47 PM

	Unknown Object (File)
	Apr 23 2025, 1:14 PM

	Unknown Object (File)
	Apr 21 2025, 2:38 AM

	Unknown Object (File)
	Apr 16 2025, 11:23 AM

	Unknown Object (File)
	Apr 15 2025, 9:26 AM

View All Files

Subscribers

aran

epriestley

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rARC6769c6b17ca5: Show current config and source using arc set-config --show

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.

Closed by commit rARC6769c6b17ca5 (authored by @iodragon, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistSetConfigWorkflow.php

16 lines

Diff 17826

View Options

src/workflow/ArcanistSetConfigWorkflow.php

Show current config and source using arc set-config --showClosedPublicActions