Page MenuHomeDevCentral

Improve documentation and code types coherence
ClosedPublic

Authored by dereckson on Dec 26 2016, 02:55.
Tags
None
Referenced Files
F3114191: D769.id1943.diff
Mon, Jun 17, 07:26
F3113180: D769.id1948.diff
Mon, Jun 17, 01:22
F3110218: D769.id1948.diff
Sun, Jun 16, 14:45
F3110165: D769.id1943.diff
Sun, Jun 16, 14:41
Unknown Object (File)
Sun, Jun 16, 13:25
Unknown Object (File)
Sun, Jun 16, 13:09
Unknown Object (File)
Sun, Jun 16, 11:13
Unknown Object (File)
Sat, Jun 15, 11:59
Subscribers

Details

Summary

We use the phan static analysis tool to detect divergences
between types in the documentation and the source code.

Types given in absolute namespace notation are now prepended with \.
When the type is in the current namespace, relative notation is used.

The new PHP 7.1 pseudo-type iterable has been chosen to use with
PhabricatorStory, as there is no need for the payload sent by
Phabricator API to be an array.

This is no-op as only method signatures and documentation is updated.

Test Plan

phan | grep PhanTypeMismatch

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

More fixes, params and properties too

dereckson retitled this revision from Improve return types doc to Improve documentation and code types coherence .
dereckson edited the summary of this revision. (Show Details)
dereckson edited the test plan for this revision. (Show Details)
dereckson edited the summary of this revision. (Show Details)
dereckson added a reviewer: dereckson.
This revision is now accepted and ready to land.Dec 27 2016, 20:48
This revision was automatically updated to reflect the committed changes.