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

SerialiSe all the thingz #50

Closed
wants to merge 2 commits into from

Conversation

ibotty
Copy link

@ibotty ibotty commented Apr 28, 2016

fixes #48.

@ibotty
Copy link
Author

ibotty commented Apr 28, 2016

I did not touch the version, but of course, that's a PVP-major change.

@@ -2,7 +2,7 @@
{-# OPTIONS_GHC -fno-warn-orphans #-}
-----------------------------------------------------------------------------
-- |
-- Module : Data.Binary.Serialize.CBOR
-- Module : Data.Binary.Serialise.CBOR
-- Copyright : 2013 Simon Meier <iridcode@gmail.com>,
Copy link
Author

Choose a reason for hiding this comment

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

That looks odd, because it's the wrong module name all along...

@thoughtpolice
Copy link
Collaborator

Yeah, note the only real interface change here is in GSerialise, which actually was just added very recently. And we haven't done a Hackage release just yet - so no bump need here, I think.

@dcoutts
Copy link
Member

dcoutts commented Apr 28, 2016

https://www.youtube.com/watch?v=Mj2cXQXgyWE&t=43m06s

This is clearly the most critical problem in the library. Thanks you @ibotty for addressing it so swiftly.

@thoughtpolice
Copy link
Collaborator

We'll have to implement the protective dragon mentioned in #48 later I guess.

thoughtpolice pushed a commit that referenced this pull request May 6, 2016
cc issue #48, issue #50

Signed-off-by: Austin Seipp <austin@well-typed.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Serialise vs Serialize
3 participants