It should be also valid to have empty change under hg
ClosedPublic
Actions

Authored by • sowedance on Apr 28 2014, 10:28 PM.

Details

Reviewers

epriestley
lifeihuang

Group Reviewers

Blessed Reviewers

Commits

rARCc4ec22b140db: It should be also valid to have empty change under hg

Summary

Like git, it is also valid to have empty net change under hg. Should not throw if that is the case.

Test Plan

Tried on a hg repo.

Diff Detail

Repository

rARC Arcanist

Branch

empty

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 133
Build 133: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

• sowedance updated this revision to Diff 21084.Apr 28 2014, 10:28 PM

• sowedance retitled this revision from to It should be also valid to have empty change under hg.

• sowedance updated this object.

• sowedance edited the test plan for this revision. (Show Details)

• sowedance added reviewers: epriestley, lifeihuang.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 28 2014, 10:28 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B105: Diff 21084.Apr 28 2014, 10:30 PM

Flag seems unrelated? Other fix is fine.

src/workflow/ArcanistDiffWorkflow.php
397–401 ↗	(On Diff #21084)	Unrelated?

This revision now requires changes to proceed.Apr 29 2014, 11:49 AM

Ah I actually want to make that change but did not mean to put it in this change.
Anyway since it's already there, I'll live with it. The reason we want that is that we try to use arc
in some automated tools and sometimes the generated code could exceed the limit.

Harbormaster completed remote builds in B121: Diff 21110.Apr 29 2014, 5:21 PM