Page MenuHomeDevCentral

PhabricatorAPI: improve code documentation, scope review
ClosedPublic

Authored by dereckson on Jan 4 2016, 19:42.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 30, 01:26
Unknown Object (File)
Mon, Jun 30, 00:21
Unknown Object (File)
Mon, Jun 30, 00:01
Unknown Object (File)
Sun, Jun 29, 16:36
Unknown Object (File)
Sun, Jun 29, 15:13
Unknown Object (File)
Sat, Jun 28, 04:50
Unknown Object (File)
Thu, Jun 26, 17:23
Unknown Object (File)
Tue, Jun 17, 15:35
Subscribers
None

Details

Summary

Code documentation

  • Array type is set to 'array' and not 'Array' for @return @var and @param

Scope review

Some methods are now now private: fetchFromAPI, isCached

Test Plan

php artisan phabricator:projectsmap (test requires D165).

Diff Detail

Repository
rNOTIF Notifications center
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
private
Build Status
Buildable 206
Build 206: arc lint + arc unit

Event Timeline

dereckson retitled this revision from to Improve code documentation.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson edited the test plan for this revision. (Show Details)
dereckson retitled this revision from Improve code documentation to PhabricatorAPI: improve code documentation, scope review.
dereckson updated this object.
dereckson edited edge metadata.

As we've already two changes, let's add also the doc one.

dereckson edited edge metadata.

+one commit, Improve PhabricatorAPI code documentation

dereckson edited edge metadata.
This revision is now accepted and ready to land.Jan 4 2016, 19:46
This revision was automatically updated to reflect the committed changes.
dereckson edited edge metadata.