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

	F15332074: D7851.diff
	Fri, Mar 7, 4:29 PM

	Unknown Object (File)
	Mon, Mar 3, 3:59 PM

	Unknown Object (File)
	Wed, Feb 19, 2:28 AM

	Unknown Object (File)
	Feb 10 2025, 7:27 PM

	Unknown Object (File)
	Feb 6 2025, 5:30 AM

	Unknown Object (File)
	Jan 24 2025, 10:36 PM

	Unknown Object (File)
	Jan 24 2025, 10:41 AM

	Unknown Object (File)
	Jan 24 2025, 6:33 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