Differential D7481

Handle empty output from `hg --debug branches` in the parser
ClosedPublic
Actions

Authored by epriestley on Nov 2 2013, 7:57 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T1493: Empty repositories should show a "this repository is empty" message

Commits

rARC9310df961540: Handle empty output from `hg --debug branches` in the parser

Summary

Ref T1493. Also consolidate this a bit more.

Test Plan

Unit tests.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Is there a bug in the presentation of this diff? branches-empty.txt seems confusing - is it an empty file maybe?

It's an empty file, yes. The display seems buggy to me.

Closed by commit rARC9310df961540 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

repository/

api/

ArcanistMercurialAPI.php

19 lines

parser/

ArcanistMercurialParser.php

8 lines

__tests__/

ArcanistMercurialParserTestCase.php

4 lines

mercurial/

branches-empty.txt

Diff 16891

View Options

src/repository/api/ArcanistMercurialAPI.php

View Options

src/repository/parser/ArcanistMercurialParser.php

View Options

src/repository/parser/tests/ArcanistMercurialParserTestCase.php

View Options

src/repository/parser/tests/mercurial/branches-empty.txt

Handle empty output from `hg --debug branches` in the parserClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 16891

src/repository/api/ArcanistMercurialAPI.php

src/repository/parser/ArcanistMercurialParser.php

src/repository/parser/__tests__/ArcanistMercurialParserTestCase.php

src/repository/parser/__tests__/mercurial/branches-empty.txt

Handle empty output from `hg --debug branches` in the parser
ClosedPublic
Actions

Revision Contents
Changeset List

src/repository/parser/tests/ArcanistMercurialParserTestCase.php

src/repository/parser/tests/mercurial/branches-empty.txt