HomePhabricator

Remove the `Futures()` method

Description

Remove the Futures() method

Summary: Fixes T6829. Removes the Futures() method in favor of calling new FutureIterator($futures) explicitly. Depends on D11077, D11078 and D11079.

Test Plan: We could possibly deprecate this function instead (with phutil_deprecated).

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin, epriestley

Maniphest Tasks: T6829

Differential Revision: https://secure.phabricator.com/D11081

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Dec 30 2014, 12:39 PM
Reviewer
Blessed Reviewers
Differential Revision
D11081: Remove the `Futures()` method
Parents
rPHUe6fef2c60a35: Use `new FutureIterator` instead of `Futures`
Branches
Unknown
Tags
Unknown
Tasks
T6829: The `Futures()` function violates naming conventions