Page MenuHomePhabricator
Differential D16487

Update Diviner for array-valued `@doc-stuff` return values from DocblockParser
ClosedPublic
Actions

Authored by epriestley on Sep 2 2016, 11:53 AM.
Tags
None
Referenced Files
F15463512: D16487.id.diff
Wed, Apr 2, 12:23 AM
F15458918: D16487.diff
Mon, Mar 31, 8:16 AM
F15431308: D16487.id39665.diff
Mar 24 2025, 11:42 AM
F15423800: D16487.id39665.diff
Mar 22 2025, 6:25 PM
F15335588: D16487.diff
Mar 8 2025, 4:35 PM
Unknown Object (File)
Feb 18 2025, 11:41 AM
Unknown Object (File)
Feb 9 2025, 2:17 AM
Unknown Object (File)
Feb 9 2025, 2:17 AM
View All Files
Subscribers
None

Details

Reviewers
joshuaspence
yelirekim
chad
Maniphest Tasks
T11575: Make PhutilDocblockParser always return arrays when parsing "@doc-tags"
Commits
rP1eee75496a5a: Update Diviner for array-valued `@doc-stuff` return values from DocblockParser
Summary

Ref T11575. After D16431, the parser may return arrays.

Test Plan

Ran bin/diviner generate --clean in phabricator/ without errors. Previously, this raised some parsing errors related to getting arrays where strings were expected.

Diff Detail

Repository
rP Phabricator
Branch
divinerdoc
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13540
Build 17432: Run Core Tests
Build 17431: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 39665.Sep 2 2016, 11:53 AM
epriestley retitled this revision from to Update Diviner for array-valued `@doc-stuff` return values from DocblockParser.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, yelirekim, joshuaspence.
epriestley added a task: T11575: Make PhutilDocblockParser always return arrays when parsing "@doc-tags".
joshuaspence accepted this revision.Sep 2 2016, 12:02 PM
joshuaspence edited edge metadata.
This revision is now accepted and ready to land.Sep 2 2016, 12:02 PM
epriestley mentioned this in T11575: Make PhutilDocblockParser always return arrays when parsing "@doc-tags".Sep 2 2016, 12:24 PM
Closed by commit rP1eee75496a5a: Update Diviner for array-valued `@doc-stuff` return values from DocblockParser (authored by epriestley, committed by epriestley). · Explain WhySep 2 2016, 12:24 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diviner/
atomizer/
11 lines

Diff 39665

View Options

src/applications/diviner/atomizer/DivinerPHPAtomizer.php

Loading...
Phabricator · Built by Phacility