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

await lost on reformat #687

Closed
cknitt opened this issue Oct 16, 2022 · 0 comments · Fixed by #688
Closed

await lost on reformat #687

cknitt opened this issue Oct 16, 2022 · 0 comments · Fixed by #688
Milestone
v10.1 "The New Fe...

Comments

@cknitt
Copy link
Member

cknitt commented Oct 16, 2022

From #686 (comment):

(await (server->start))->ignore

loses the await on reformat:

server->start->ignore
@cknitt cknitt added this to the v10.1 "The New Features" milestone Oct 16, 2022
cristianoc added a commit that referenced this issue Oct 16, 2022
@cristianoc
Fix issue where nested pipe discards await
cd0a368 
Fixes #687

Certain nested patterns for binary operators are printed "manually" so the special printing for await is not used.
@cristianoc cristianoc mentioned this issue Oct 16, 2022
Merged
@cristianoc cristianoc mentioned this issue Oct 21, 2022
Closed
57 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v10.1 "The New Features"
Development

Successfully merging a pull request may close this issue.

Fix issue where printing nested pipe discards await rescript-lang/syntax
1 participant
@cknitt