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)
Fri, Jul 18, 20:48
Unknown Object (File)
Tue, Jul 15, 03:08
Unknown Object (File)
Mon, Jul 14, 09:12
Unknown Object (File)
Sun, Jul 13, 06:42
Unknown Object (File)
Fri, Jul 11, 15:12
Unknown Object (File)
Wed, Jul 9, 19:03
Unknown Object (File)
Sat, Jul 5, 19:15
Unknown Object (File)
Sat, Jul 5, 19:14
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 207
Build 207: 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.