You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
((match foo with | Bar -> bar | Baz -> baz) : string)
Is now formatted like this:
(match foo with
| Bar -> bar
| Baz -> baz
: string)
It used to leave the inner parentheses intact, to clearly associate the type annotation with match instead of baz.
On its own, the new code would probably be fine, as the indentation makes the association pretty clear. However, with our current use of ocp-indent, the : string gets indented flush with baz. So until we're ready to ditch ocp-indent, we'd like to keep the extra parentheses.
Edit: I only checked this with --profile=janestreet, and I'm only requesting the change in that profile.
The text was updated successfully, but these errors were encountered:
The following code:
Is now formatted like this:
It used to leave the inner parentheses intact, to clearly associate the type annotation with
match
instead ofbaz
.On its own, the new code would probably be fine, as the indentation makes the association pretty clear. However, with our current use of
ocp-indent
, the: string
gets indented flush withbaz
. So until we're ready to ditch ocp-indent, we'd like to keep the extra parentheses.Edit: I only checked this with
--profile=janestreet
, and I'm only requesting the change in that profile.The text was updated successfully, but these errors were encountered: