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, Jan 23, 12:51 AM

	Unknown Object (File)
	Sat, Jan 18, 2:05 AM

	Unknown Object (File)
	Fri, Dec 27, 12:25 PM

	Unknown Object (File)
	Dec 17 2024, 6:20 AM

	Unknown Object (File)
	Dec 14 2024, 6:47 AM

	Unknown Object (File)
	Dec 3 2024, 11:24 AM

	Unknown Object (File)
	Dec 2 2024, 9:38 AM

	Unknown Object (File)
	Nov 26 2024, 9:18 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