Improve some messaging in `arc which` for autodetection of repositories
ClosedPublic
Actions

Authored by epriestley on Jan 28 2014, 2:00 AM.

Tags

None

Referenced Files

	F19067532: D8081.id18283.diff
	Sun, Nov 30, 8:29 AM

	F19060086: D8081.id18286.diff
	Sat, Nov 29, 8:13 AM

	F18997511: D8081.diff
	Thu, Nov 20, 1:31 PM

	F18887380: D8081.id18283.diff
	Nov 7 2025, 5:57 AM

	F18827363: D8081.id18286.diff
	Oct 24 2025, 9:40 AM

	F18808021: D8081.diff
	Oct 19 2025, 4:44 AM

	F18432576: D8081.id18286.diff
	Aug 31 2025, 6:49 AM

	F18432303: D8081.id18286.diff
	Aug 31 2025, 6:29 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan
talshiri

Commits

rARC7116659289a4: Improve some messaging in `arc which` for autodetection of repositories

Summary

The modern name for the config is "project.name".
Missing parameter in a pht().
When the value is set, but not valid, we gave you a misleading error message.

Test Plan

Ran arc which.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

talshiri added inline comments.Jan 28 2014, 3:03 AM

src/workflow/ArcanistBaseWorkflow.php
1661	This slightly lies though, doesn't it? getProjectID() falls back to fetching project_id

epriestley added inline comments.Jan 28 2014, 3:05 AM

src/workflow/ArcanistBaseWorkflow.php
1661	"project_id" is a deprecated spelling of "project.name"; "project.id" (`.` vs `_`) was never correct, I just neglected to fix it after renaming `project.id` -> `project.name` mid-patch.

talshiri accepted this revision.Jan 28 2014, 3:12 AM

Closed by commit rARC7116659289a4 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistBaseWorkflow.php

17 lines

Diff 18286

View Options

src/workflow/ArcanistBaseWorkflow.php

Improve some messaging in `arc which` for autodetection of repositoriesClosedPublicActions