This is an archive of the discontinued LLVM Phabricator instance.

Include phabricator.uri in .arcconfig
ClosedPublic

Authored by phosek on Jan 17 2020, 2:41 PM.

Reviewers

Commits

rG9ad044a38c00: Include phabricator.uri in .arcconfig

Summary

conduit_uri was renamed to phabricator.uri and git-phab fails to load
.arcconfig without this field.

lgtm

This revision is now accepted and ready to land.Jan 22 2020, 3:24 PM

By git-phab do you mean new versions of arcanist? I don't see a git-phab tool in tree.

In D72960#1835473, @smeenai wrote:

I'm assuming git-phab is https://pypi.org/project/git-phab/

Yes, that ^. Someone mentioned it in the Phab/GitHub PR email thread as an alternative to arcanist so I'm giving it a try.

This revision was automatically updated to reflect the committed changes.

Path

Size

1 line

	{			{
				"phabricator.uri" : "https://reviews.llvm.org/",
	"repository.callsign" : "G",			"repository.callsign" : "G",
	"conduit_uri" : "https://reviews.llvm.org/"			"conduit_uri" : "https://reviews.llvm.org/"
	}			}