Fix sanity checks for patch workflow
Audited1f2b51c4a396
Actions

Description

Fix sanity checks for patch workflow

Summary:
Ref T7604. Ref T8307. This was broken in D12962 because I only tested arc patch --arcbundle. Furthermore, this particular sanity check doesn't actually do anything now (see T8307).

Test Plan:
Ran arc patch --nobranch D12971 successfully.

Auditors: epriestley

Details

Auditors

epriestley

Group Auditors

Cowboy Commits

Provenance

joshuaspence	Authored on
joshuaspence	Pushed on May 25 2015, 8:42 AM

Parents

rARCbe9dd352be58: Remove call to "arcanist.projectinfo" from `arc export`

Branches

Unknown

Tags

Unknown

Tasks

T8307: `arc patch` applies diffs from another repository
T7604: Remove "Arcanist Projects"

Event Timeline

joshuaspence committed rARC1f2b51c4a396: Fix sanity checks for patch workflow (authored by joshuaspence).May 25 2015, 8:41 AM

joshuaspence added a task: T7604: Remove "Arcanist Projects".

joshuaspence added a task: T8307: `arc patch` applies diffs from another repository.

joshuaspence added auditors: epriestley.

Herald added auditors: Fakhra.Aftab, Cowboy Commits. · View Herald TranscriptMay 25 2015, 8:42 AM

Cool. I don't think T8307 impacts too many people, and its nice to see a tracking task. :D

epriestley accepted this commit.Jun 15 2015, 12:48 PM

Changes (1)

Path

Size

src/

workflow/

ArcanistPatchWorkflow.php

rARC1f2b51c4a396

View Options

src/workflow/ArcanistPatchWorkflow.php