Paths

Table of Contentst

Differential D11149

Remove an unused class
ClosedPublic
Actions

Authored by joshuaspence on Jan 2 2015, 1:31 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP1b80f6e09d71: Remove an unused class

Summary

This class is unused after D6679.

Test Plan

grep

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 3485
Build 3492: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 26761.Jan 2 2015, 1:31 PM

joshuaspence retitled this revision from to Remove an unused class.

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 2 2015, 1:31 PM

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

epriestley accepted this revision.Jan 2 2015, 4:46 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jan 2 2015, 4:46 PM

Closed by commit rP1b80f6e09d71: Remove an unused class (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 2 2015, 10:11 PM

This revision was automatically updated to reflect the committed changes.

joshuaspence mentioned this in T6853: "Open Revisions Affecting These Files" should weight severity of changes.Jan 2 2015, 10:14 PM

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

herald/

engine/

exception/

HeraldInvalidFieldException.php

Diff 26761

View Options

src/__phutil_library_map__.php