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

allow derivingVia using quantified constraints to be expressible! #49

Merged
merged 1 commit into from
Jan 31, 2020

Conversation

cartazio
Copy link
Contributor

No description provided.

@ekmett ekmett merged commit 402d479 into ekmett:master Jan 31, 2020
@cartazio
Copy link
Contributor Author

https://gitlab.haskell.org/ghc/ghc/issues/17767#note_251123 for how this came about

and

newtype MyDouble = MYD Double

type MGetCoerT m = (forall p q. (MonadGet m, Coercible p q) => Coercible (m p) (m q) :: Constraint)
deriving via Double instance MGetCoerT m => Serial MyDouble

@cartazio
Copy link
Contributor Author

so that folks can see how to write stuff

@cartazio
Copy link
Contributor Author

@RyanGlScott @ekmett would this be a minor or Major bump? and also a friend of ours would like it on hackage so he can use it ;)

@ekmett
Copy link
Owner

ekmett commented Jan 31, 2020

I'd probably major bump it to be safe.

@cartazio
Copy link
Contributor Author

yeah, that extra constraint might impact someone somehwere

@cartazio
Copy link
Contributor Author

@RyanGlScott :)

@cartazio
Copy link
Contributor Author

cartazio commented Feb 1, 2020 via email

@cartazio
Copy link
Contributor Author

cartazio commented Feb 1, 2020 via email

RyanGlScott added a commit that referenced this pull request Feb 1, 2020
@RyanGlScott
Copy link
Collaborator

I'll plan to make a new Hackage release of bytes sometime next week alongside other @ekmett libraries that need major version bumps (namely, ad, constraints, lens, and linear).

@RyanGlScott
Copy link
Collaborator

I've released bytes-0.17 to Hackage with these changes.

@cartazio
Copy link
Contributor Author

cartazio commented Feb 3, 2020 via email

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.

4 participants