Skip to content
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.

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

Implement Read instance for Snowflake #35

Merged
merged 3 commits into from
Mar 26, 2020

Conversation

rexim
Copy link
Contributor

@rexim rexim commented Mar 26, 2020

I'm not sure why this instance was not implemented, but I really need it to simplify some of the serialization I do in my bot that uses your beautiful library (thank you so much for implementing it <3).

Please let me know if there are any problems with my PR. Thanks!

I'm not sure why this instance was not implemented, but I really
needed to simplify some of the serialization I do in my bot that uses
your beautiful library. (thank you so much for implementing it <3).
Copy link
Collaborator

@aquarial aquarial left a comment

Choose a reason for hiding this comment

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

Cool to see, I'm glad you're getting good use out of it

src/Discord/Internal/Types/Prelude.hs Show resolved Hide resolved
@aquarial aquarial merged commit eb1d069 into discord-haskell:master Mar 26, 2020
@rexim rexim deleted the read-snowflake branch March 26, 2020 22:12
@aquarial
Copy link
Collaborator

aquarial commented Mar 26, 2020

I'm going to make sure the project will build on stackage lts-15.5, and then push this change as version 1.5.0 of the library.

Usually can just loosen the .cabal restrictions

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.

2 participants