⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Trying the pipe operator#1

Draft
thgs wants to merge 3 commits intomasterfrom
pipe-op
Draft

Trying the pipe operator#1
thgs wants to merge 3 commits intomasterfrom
pipe-op

Conversation

@thgs
Copy link
Owner

@thgs thgs commented Mar 11, 2025

This is very minimal changes and not everywhere yet.
There's a lot uncommitted in my local branch at the moment.

Comment on lines +118 to +120
$do = fn ()
=> null |> $fa
|> partial (null |> $this);
Copy link
Owner Author

Choose a reason for hiding this comment

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

I think this would look better like

Suggested change
$do = fn ()
=> null |> $fa
|> partial (null |> $this);
$do = fn () => $fa() |> partial ($this());


// todo: could add a type check here? that return type is indeed m b ?
return (partial ($f) ($x))
return (null |> $action |> partial ($f))
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
return (null |> $action |> partial ($f))
return ($action() |> partial ($f))

@thgs thgs changed the title Trying the pipe op Trying the pipe operator Mar 11, 2025
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.

1 participant