Paths

Table of Contentst

Differential D13377

Render lint results as Harbormaster lint messages
ClosedPublic
Actions

Authored by epriestley on Jun 21 2015, 5:14 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T8095: Let Harbormaster own Lint and Unit results

Commits

Restricted Diffusion Commit
rPe618b672d233: Render lint results as Harbormaster lint messages

Summary

Ref T8095. Render lint results in a future-ready way.

This makes the renderer accept HarbormasterBuildLintMessage objects. If we have legacy data instead, it converts it into HarbormasterBuildLintMessage objects.

Design is a bit rough but will be cleaned up later after T7739.

This moves away from "postponed linters", which are obsolete after Harbormaster (and were only ever used by Facebook).

Test Plan

Screen Shot 2015-06-21 at 10.09.34 AM.png (291×1 px, 61 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 32389.Jun 21 2015, 5:14 PM

epriestley retitled this revision from to Render lint results as Harbormaster lint messages.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T8095: Let Harbormaster own Lint and Unit results.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 21 2015, 5:14 PM

epriestley mentioned this in D13378: Render unit results as Harbormaster unit messages.Jun 21 2015, 5:39 PM

btrahan accepted this revision.Jun 22 2015, 4:59 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Jun 22 2015, 4:59 PM

epriestley mentioned this in T8089: Unprototype Harbormaster (v1).Jun 22 2015, 9:21 PM

Closed by commit rPe618b672d233: Render lint results as Harbormaster lint messages (authored by epriestley, committed by epriestley). · Explain WhyJun 23 2015, 5:24 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in rPb72deb332ca7: Render unit results as Harbormaster unit messages.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

differential/

customfield/

DifferentialLintField.php

265 lines

harbormaster/

view/

HarbormasterLintPropertyView.php

78 lines

view/

phui/

PHUIStatusItemView.php

1 line

Diff 32455

View Options

src/__phutil_library_map__.php

View Options

src/applications/differential/customfield/DifferentialLintField.php

View Options

src/applications/harbormaster/view/HarbormasterLintPropertyView.php

View Options

src/view/phui/PHUIStatusItemView.php

Render lint results as Harbormaster lint messagesClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 32455

src/__phutil_library_map__.php

src/applications/differential/customfield/DifferentialLintField.php

src/applications/harbormaster/view/HarbormasterLintPropertyView.php

src/view/phui/PHUIStatusItemView.php

Render lint results as Harbormaster lint messages
ClosedPublic
Actions

Revision Contents
Changeset List