Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

Whitespace with new fast pipes? #93

Open
Anahkiasen opened this issue Nov 16, 2018 · 0 comments
Open

Whitespace with new fast pipes? #93

Anahkiasen opened this issue Nov 16, 2018 · 0 comments

Comments

@Anahkiasen
Copy link

Anahkiasen commented Nov 16, 2018

I updated refmt to 3.3.3 and it modified one my blocks in the following way

-  expenditure
-  |. getExpenditureAccounts
-  |. Belt.List.fromArray
-  |. Belt.List.getBy(id => id == accountId)
-  |. Option.isSome;
+  expenditure->getExpenditureAccounts->Belt.List.fromArray->(Belt.List.getBy(id => id == accountId))->Option.isSome;

Now I'm aware of the change from |. to -> but does the whitespace need to be discarded around the operator for it to work? I don't particularly find the updated code easier to read, quite the opposite :/

Could it not simply reformat it to this?

expenditure
-> getExpenditureAccounts
-> Belt.List.fromArray
-> Belt.List.getBy(id => id == accountId)
-> Option.isSome;
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant