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

	Unknown Object (File)
	Thu, May 2, 6:56 PM

	Unknown Object (File)
	Fri, Apr 12, 1:47 PM

	Unknown Object (File)
	Fri, Apr 12, 11:48 AM

	Unknown Object (File)
	Fri, Apr 12, 11:48 AM

	Unknown Object (File)
	Fri, Apr 12, 10:55 AM

	Unknown Object (File)
	Mar 25 2024, 1:13 AM

	Unknown Object (File)
	Feb 15 2024, 2:36 AM

	Unknown Object (File)
	Feb 15 2024, 2:36 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