-
Notifications
You must be signed in to change notification settings - Fork 10
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
Separate msgpack types from serialization details [2/2] #43
Conversation
Signed-off-by: Anton Kondrachkov <SX91@protonmail.com>
data-msgpack.cabal
Outdated
@@ -1,5 +1,5 @@ | |||
name: data-msgpack | |||
version: 0.0.9 | |||
version: 0.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this a minor version bump? Why not 0.0.10?
Please enable the checkbox "Allow edits from maintainers." on the bottom right. |
@iphydf It's already checked. And I've fixed version to be a patch bump. |
Any idea why appveyor is failing? |
Review status: 0 of 14 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed. Comments from Reviewable |
@SX91 you have push access to this repo, so once it's ready, feel free to merge. |
So, what about actually merging this? |
Reviewed 13 of 14 files at r1, 1 of 1 files at r5. Comments from Reviewable |
This is the second part of TokTok/hs-msgpack-types#1
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)