Skip to content

Add option to not insert a space before record braces#26

Merged
georgefst merged 2 commits intomasterfrom
record-space
Sep 24, 2020
Merged

Add option to not insert a space before record braces#26
georgefst merged 2 commits intomasterfrom
record-space

Conversation

@georgefst
Copy link
Collaborator

Closes one of #19.

@georgefst
Copy link
Collaborator Author

georgefst commented Sep 20, 2020

I opted to keep the space in data type declarations. It just feels weird somehow:

data Foo = Foo {fooX :: Int, fooY :: Int}
data Foo = Foo{fooX :: Int, fooY :: Int}

Anyway, in general, the major reason for preferring not to have the space, is to underline the fact that record update/construction binds even more tightly than function composition, and that isn't relevant here.

I'm open to changing it if anyone has an opinion.

Edit: Come to think of it, I don't think I ever define record types on a single line, so this shouldn't actually bother me.

@georgefst georgefst merged commit c49c79e into master Sep 24, 2020
@brandonchinn178 brandonchinn178 deleted the record-space branch August 10, 2021 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant