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)
	Sat, Dec 28, 3:09 PM

	Unknown Object (File)
	Sun, Dec 22, 1:27 PM

	Unknown Object (File)
	Fri, Dec 20, 4:55 PM

	Unknown Object (File)
	Tue, Dec 17, 12:07 PM

	Unknown Object (File)
	Sun, Dec 15, 6:37 AM

	Unknown Object (File)
	Thu, Dec 12, 4:06 PM

	Unknown Object (File)
	Tue, Dec 10, 4:31 PM

	Unknown Object (File)
	Dec 8 2024, 2:26 PM

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