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

Providing Pipe Function see #141 #200

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

tzkmx
Copy link

@tzkmx tzkmx commented Nov 26, 2019

In order to support pipe function, I'm contributing my proposal to get this long before reaching 2.0.

Copy link
Owner

@lstrojny lstrojny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s a great addition to functional, thank you!

src/Functional/Pipe.php Outdated Show resolved Hide resolved
src/Functional/Pipe.php Outdated Show resolved Hide resolved
tests/Functional/PipeTest.php Show resolved Hide resolved
@lstrojny
Copy link
Owner

Could you change to PHPUnit mocks, then we are ready to go

@tzkmx tzkmx requested a review from lstrojny August 1, 2020 05:26
Base automatically changed from master to main March 5, 2021 16:00
@simPod
Copy link

simPod commented Aug 22, 2021

Hi, are we going to pursue this?

Copy link

@simPod simPod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation is missing

@allenisalai
Copy link

Is there anything I can do to help get this PR merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants