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

	F15433787: D8081.id.diff
	Tue, Mar 25, 12:39 AM

	F15433042: D8081.id18286.diff
	Mon, Mar 24, 9:04 PM

	F15429685: D8081.diff
	Mon, Mar 24, 3:37 AM

	Unknown Object (File)
	Feb 22 2025, 10:52 PM

	Unknown Object (File)
	Feb 19 2025, 10:39 PM

	Unknown Object (File)
	Feb 12 2025, 11:46 AM

	Unknown Object (File)
	Jan 30 2025, 8:08 AM

	Unknown Object (File)
	Jan 30 2025, 7:10 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