Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change to use functional API instead of static Util class #6

Merged
merged 2 commits into from
Oct 20, 2021

Conversation

clue
Copy link
Member

@clue clue commented Oct 20, 2021

The raising of the dead part 5.

This changeset updates the code base to use a functional API instead of a static Util class.

// old
React\Async\Util::parallel(…);
React\Async\Util::series(…);
React\Async\Util::waterfall(…);

// new
React\Async\parallel(…);
React\Async\series(…);
React\Async\waterfall(…);

Refs clue/reactphp-block#13, reactphp/promise#12 and others
Builds on top of #2, #3, #4 and #5

@clue clue added new feature New feature or request BC break labels Oct 20, 2021
@clue clue added this to the v2.0.0 milestone Oct 20, 2021
@clue clue requested a review from WyriHaximus October 20, 2021 05:37
@WyriHaximus WyriHaximus merged commit e0a97b1 into reactphp:main Oct 20, 2021
@clue clue deleted the functions branch November 20, 2021 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BC break new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants