Differential D21342

Verify remotes ("paths") in Mercurial during "arc land"
ClosedPublic
Actions

Authored by epriestley on Jun 9 2020, 1:35 PM.

Tags

None

Referenced Files

	F13224662: D21342.diff
	Sun, May 19, 9:55 AM

	F13209233: D21342.id.diff
	Thu, May 16, 10:46 PM

	F13204477: D21342.diff
	Wed, May 15, 12:37 AM

	F13187440: D21342.diff
	Sat, May 11, 4:31 AM

	Unknown Object (File)
	Tue, May 7, 7:38 AM

	Unknown Object (File)
	Mon, May 6, 8:48 PM

	Unknown Object (File)
	Sat, May 4, 10:56 AM

	Unknown Object (File)
	Fri, May 3, 7:22 AM

View All 67 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13546: Modernize the "arc land" workflow
Commits: rARC1bb054ef47a1: Verify remotes ("paths") in Mercurial during "arc land"

Summary

Ref T13546. Parse "hg paths" and validate that the remotes "arc land" plans to interact with actually exist.

Test Plan

Ran "arc land" with good and bad "--into-remote" and "--onto-remote" arguments, got sensible validation behavior.

Diff Detail

Repository

Branch

hg1

Lint

Lint Passed

Severity	Location	Code	Message
Advice	src/land/engine/ArcanistMercurialLandEngine.php:142	XHP16	TODO Comment
Advice	src/land/engine/ArcanistMercurialLandEngine.php:287	XHP16	TODO Comment
Advice	src/repository/api/ArcanistMercurialAPI.php:905	XHP16	TODO Comment
Advice	src/workflow/ArcanistWorkflow.php:2039	XHP16	TODO Comment

Unit

Tests Passed

Build Status

Buildable 24627
Build 33949: Run Core Tests
Build 33948: arc lint + arc unit

Event Timeline

epriestley created this revision.Jun 9 2020, 1:35 PM

Harbormaster completed remote builds in B24627: Diff 50806.Jun 9 2020, 1:36 PM

epriestley requested review of this revision.Jun 9 2020, 1:36 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 10 2020, 5:27 PM

Closed by commit rARC1bb054ef47a1: Verify remotes ("paths") in Mercurial during "arc land" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC1bb054ef47a1: Verify remotes ("paths") in Mercurial during "arc land".

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

land/

engine/

ArcanistMercurialLandEngine.php

28 lines

repository/

api/

ArcanistMercurialAPI.php

6 lines

ArcanistRepositoryAPI.php

9 lines

marker/

ArcanistRepositoryMarkerQuery.php

28 lines

query/

ArcanistRepositoryQuery.php

35 lines

remote/

ArcanistMercurialRepositoryRemoteQuery.php

45 lines

ArcanistRemoteRef.php

41 lines

ArcanistRepositoryRemoteQuery.php

31 lines

workflow/

ArcanistWorkflow.php

2 lines

Diff 50806

src/__phutil_library_map__.php

Loading...

src/land/engine/ArcanistMercurialLandEngine.php

Loading...

src/repository/api/ArcanistMercurialAPI.php

Loading...

src/repository/api/ArcanistRepositoryAPI.php

Loading...

src/repository/marker/ArcanistRepositoryMarkerQuery.php

Loading...

src/repository/query/ArcanistRepositoryQuery.php

Loading...

src/repository/remote/ArcanistMercurialRepositoryRemoteQuery.php

Loading...

src/repository/remote/ArcanistRemoteRef.php

Loading...

src/repository/remote/ArcanistRepositoryRemoteQuery.php

Loading...

src/workflow/ArcanistWorkflow.php

Loading...