Skip to content

Example of broken formatting for foo(a,_) in uncurried mode. #6148

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 5 commits into from
Apr 14, 2023
Merged

Example of broken formatting for foo(a,_) in uncurried mode. #6148

merged 5 commits into from
Apr 14, 2023

Conversation

cristianoc
Copy link
Collaborator

See #6146

@cristianoc cristianoc mentioned this pull request Apr 14, 2023
Closed
@DZakh
Copy link
Member

DZakh commented Apr 14, 2023

Input:

let concatStrings = (string1, string2) => string1 ++ string2
Ok("_")->Belt.Result.map(concatStrings(_, "foo"))

Formatted:

let concatStrings = (string1, string2) => string1 ++ string2
Ok("_")->Belt.Result.map(() => concatStrings(_, "foo"))

@cristianoc cristianoc merged commit cde772a into master Apr 14, 2023
@cristianoc cristianoc deleted the uncurried_placeholder_app branch April 14, 2023 15:02
DZakh
DZakh reviewed Apr 14, 2023
View reviewed changes
cristianoc added a commit that referenced this pull request Apr 14, 2023
@cristianoc
Fix PR 6148
bfa8c49 
Fix #6148
@cristianoc cristianoc mentioned this pull request Apr 14, 2023
Merged
cristianoc added a commit that referenced this pull request Apr 14, 2023
@cristianoc
Fix PR 6148
379b4ae 
Fix #6148
@fhammerschmidt fhammerschmidt mentioned this pull request Oct 2, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DZakh DZakh DZakh left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cristianoc @DZakh