Homec4science

Show lint/unit failure explanation in Phabricator

Authored by Natthu Bharambe <natthu@fb.com> on Feb 24 2012, 23:26.

Description

Show lint/unit failure explanation in Phabricator

Summary:
Tweaked lint/unit field specifications to introduce the failure
explanation read from arc:[lint|unit]-excuse.

Task ID: #

Blame Rev:

Test Plan:
Create dumb diffs with errors - run modified 'arc' and change
conduit_url to http://phabricator.dev1020.facebook.com/api/ - verified
that explanation shows up with proper formatting.

Revert Plan:

Tags:

Reviewers: epriestley, nh

Reviewed By: epriestley

CC: Girish, akramer, blair, aran, epriestley, andreygoder

Differential Revision: https://secure.phabricator.com/D1689

Details

Committed
Natthu Bharambe <natthu@fb.com>Feb 25 2012, 00:28
Pushed
aubortJan 31 2017, 17:16
Parents
rPH89128a70d572: Remove references to nonexistent PhabricatorOAuthProviderPhabricator
Branches
Unknown
Tags
Unknown

Event Timeline

Natthu Bharambe <natthu@fb.com> committed rPHed1928eee267: Show lint/unit failure explanation in Phabricator (authored by Natthu Bharambe <natthu@fb.com>).Feb 25 2012, 00:28