Skip to content

Treat the forward pipe (|>) as a monadic operator - #322

Merged
redianthus merged 1 commit into
OCamlPro:masterfrom
nberth:piping-as-monadicop
Jun 29, 2025
Merged

redianthus merged 1 commit into
OCamlPro:masterfrom
nberth:piping-as-monadicop

Conversation

@nberth

@nberth nberth commented Mar 22, 2023

Copy link
Copy Markdown
Contributor

Before this change the pipe was treated like standard binary operators:

let f x = x
          |> f

I think it makes more sense to treat it as a monadic:

let f x = x
  |> f

@redianthus
redianthus merged commit a9e69f7 into OCamlPro:master Jun 29, 2025
NathanReb added a commit to NathanReb/opam-repository that referenced this pull request Oct 1, 2025
CHANGES:

+ Fix detection of let vs letin after a struct (OCamlPro/ocp-indent#324, @Julow)
+ Treat `|>` as a monadic operator (OCamlPro/ocp-indent#322, @nberth)
+ Prevent Args out of range error in ocp-indent-buffer (OCamlPro/ocp-indent#327, @bcc32)
+ Fix "Missing ‘lexical-binding’ cookie" warning in Emacs (OCamlPro/ocp-indent#329, @nojb)
+ Fix a bug where ocp-indent translate LF to CRLF when run on Windows instead
  of preserving the input's newlines. (OCamlPro/ocp-indent#334, @nojb)
+ Add `ocp-indent-gen-rules` to support `dune fmt` like workflow with ocp-indent
  (OCamlPro/ocp-indent#333, @NathanReb)
nojb added a commit to LexiFi/ocp-indent that referenced this pull request Feb 9, 2026
Release 1.9.0

CHANGES:

+ Fix detection of let vs letin after a struct (OCamlPro#324, @Julow)
+ Treat `|>` as a monadic operator (OCamlPro#322, @nberth)
+ Prevent Args out of range error in ocp-indent-buffer (OCamlPro#327, @bcc32)
+ Fix "Missing ‘lexical-binding’ cookie" warning in Emacs (OCamlPro#329, @nojb)
+ Fix a bug where ocp-indent translate LF to CRLF when run on Windows instead
  of preserving the input's newlines. (OCamlPro#334, @nojb)
+ Add `ocp-indent-gen-rules` to support `dune fmt` like workflow with ocp-indent
  (OCamlPro#333, @NathanReb)
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.

2 participants