Skip to content

Commit

Permalink
Merge pull request #220 from clue-labs/no-promisor
Browse files Browse the repository at this point in the history
[RFC] Remove unneeded `PromisorInterface`
  • Loading branch information
WyriHaximus authored May 31, 2022
2 parents 8485913 + 55f5d38 commit 4bc784e
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 21 deletions.
7 changes: 0 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ Table of Contents
* [some()](#some)
* [map()](#map)
* [reduce()](#reduce)
* [PromisorInterface](#promisorinterface)
4. [Examples](#examples)
* [How to use Deferred](#how-to-use-deferred)
* [How promise forwarding works](#how-promise-forwarding-works)
Expand Down Expand Up @@ -486,12 +485,6 @@ promises and/or values, and `$reduceFunc` may return either a value or a
promise, *and* `$initialValue` may be a promise or a value for the starting
value.

### PromisorInterface

The `React\Promise\PromisorInterface` provides a common interface for objects
that provide a promise. `React\Promise\Deferred` implements it, but since it
is part of the public API anyone can implement it.

Examples
--------

Expand Down
2 changes: 1 addition & 1 deletion src/Deferred.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

namespace React\Promise;

final class Deferred implements PromisorInterface
final class Deferred
{
private $promise;
private $resolveCallback;
Expand Down
13 changes: 0 additions & 13 deletions src/PromisorInterface.php

This file was deleted.

0 comments on commit 4bc784e

Please sign in to comment.