Paths

Table of Contentst

Remove unused class
ClosedPublic
Actions

Authored by joshuaspence on Jan 1 2015, 1:32 PM.

Details

Reviewers

epriestley
chad

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP9ddc768bb648: Remove unused class

Summary

This class is unused after D4058. maybe let's remove it?

Test Plan

grep

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 3455
Build 3462: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 26706.Jan 1 2015, 1:32 PM

joshuaspence retitled this revision from to Remove unused class.

joshuaspence updated this object.

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

joshuaspence added reviewers: epriestley, chad.

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

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

epriestley accepted this revision.Jan 1 2015, 2:52 PM

epriestley edited edge metadata.

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

Closed by commit rP9ddc768bb648: Remove unused class (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 1 2015, 11:07 PM

This revision was automatically updated to reflect the committed changes.

joshuaspence mentioned this in D11170: Remove unused `PhabricatorMainMenuGroupView` class.Jan 3 2015, 4:11 AM

joshuaspence mentioned this in rPdffdb4ab8e72: Remove unused `PhabricatorMainMenuGroupView` class.Jan 3 2015, 12:52 PM

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

view/

page/

menu/

PhabricatorMainMenuIconView.php

Diff 26706

View Options

src/__phutil_library_map__.php