Support formatting of sum types? #291

Open
opened 2020-04-03 18:51:14 +02:00 by vaclavsvejcar · 2 comments
vaclavsvejcar commented 2020-04-03 18:51:14 +02:00 (Migrated from github.com)

I'm really happy with data record types formatting in Brittany (as implemented in #161 - thank you very much @eborden), but it looks that we still don't have support for formatting sum types, such as for

data RunMode
  = Add
  | Drop
  | Replace
  deriving (Eq, Show)

Is there any plan to add support for this in recent future?

I'm really happy with data record types formatting in _Brittany_ (as implemented in #161 - thank you very much @eborden), but it looks that we still don't have support for formatting sum types, such as for ```haskell data RunMode = Add | Drop | Replace deriving (Eq, Show) ``` Is there any plan to add support for this in recent future?
eborden commented 2020-04-06 16:52:48 +02:00 (Migrated from github.com)

@vaclavsvejcar I have a branch with some of the work for this on it. I just haven't been able to get back to it. I'll get it back onto my todo list. It is something productive I can get done during social distancing.

@vaclavsvejcar I have a branch with some of the work for this on it. I just haven't been able to get back to it. I'll get it back onto my todo list. It is something productive I can get done during social distancing.
eborden commented 2020-04-10 18:09:08 +02:00 (Migrated from github.com)

Refreshed my old work and opened a PR over here. There is work left to do, but hopefully this gets this issue moving.

https://github.com/lspitzner/brittany/pull/294

Refreshed my old work and opened a PR over here. There is work left to do, but hopefully this gets this issue moving. https://github.com/lspitzner/brittany/pull/294
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: hexagoxel/brittany#291
There is no content yet.