muesli4 / table-layout

Layout data in grids and pretty tables. Provides a lot of tools to get the cell formatting right (positional alignment, alignment on specific characters and limiting of cell width)..
BSD 3-Clause "New" or "Revised" License
37 stars 11 forks source link

base bound too low #1

Closed ocharles closed 6 years ago

ocharles commented 6 years ago

Could your base upper bound be increased? 4.11 is out now.

muesli4 commented 6 years ago

I will publish an update in a moment. I will still keep an upper bound. Is this the preferred way?

ocharles commented 6 years ago

Hi, yes you need an upper bound. However it turns out you're not actually compatible with GHC 8.4 , so I wouldn't release a new version until those problems are fixed.

On Sat, 14 Jul 2018, 2:51 pm Moritz Bruder, notifications@github.com wrote:

Closed #1 https://github.com/muesli4/table-layout/issues/1.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/muesli4/table-layout/issues/1#event-1733877288, or mute the thread https://github.com/notifications/unsubscribe-auth/AABRjsRKonIylsVg7VzWdMieFGkzdcAsks5uGfdVgaJpZM4VPAC2 .

muesli4 commented 6 years ago

I've compiled and tested with GHC 8.4.3 and it seems to work. (I added Semigroup instances.)

ocharles commented 6 years ago

Ah, perfect! It was those missing instances I hit

On Sat, 14 Jul 2018, 5:19 pm Moritz Bruder, notifications@github.com wrote:

I've compiled and tested with GHC 8.4.3 and it seems to work. (I added Semigroup instances.)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/muesli4/table-layout/issues/1#issuecomment-405033685, or mute the thread https://github.com/notifications/unsubscribe-auth/AABRjkCi6rNAaPlCED-ri9kd5xp3YwAlks5uGhoVgaJpZM4VPAC2 .