Skip to content

Fix regression with multiline tuple#162

Merged
brandonchinn178 merged 1 commit intomasterfrom
tuple-multiline
Apr 4, 2022
Merged

Fix regression with multiline tuple#162
brandonchinn178 merged 1 commit intomasterfrom
tuple-multiline

Conversation

@brandonchinn178
Copy link
Copy Markdown
Collaborator

@brandonchinn178 brandonchinn178 commented Apr 4, 2022 •
edited
Loading

Fixes #148

See issue for how this broke.

Tested with the following script:

x =
  [
    ( "foo"
    , handle onError $ do
        print "asdf"
        doFoo
    )
  ]

Cherry picked to upstream: tweag/ormolu#868

@brandonchinn178 brandonchinn178 changed the title Fix multiline tuple Fix regression with multiline tuple Apr 4, 2022
georgefst
georgefst approved these changes Apr 4, 2022
View reviewed changes
Copy link
Copy Markdown
Collaborator

@georgefst georgefst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great sleuthing!

@brandonchinn178 brandonchinn178 merged commit 45d41df into master Apr 4, 2022
@brandonchinn178 brandonchinn178 deleted the tuple-multiline branch April 4, 2022 16:28
brandonchinn178 added a commit that referenced this pull request Apr 4, 2022
@brandonchinn178
Update changelog for #162
8904138
brandonchinn178 added a commit that referenced this pull request Apr 4, 2022
@brandonchinn178
Merge pull request #163 from fourmolu/changelog
9c2702f 
Update changelog for #162
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@georgefst georgefst georgefst approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Regression: fourmolu-0.5.0.0 deindents multiline tuples

2 participants

@brandonchinn178 @georgefst