-
Notifications
You must be signed in to change notification settings - Fork 479
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
HasBlueprintSchema and HasBlueprintDefinition instances #6271
Conversation
da90e6b
to
7046453
Compare
7057c67
to
d5afe39
Compare
a161078
to
db5cabb
Compare
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.
This looks largely fine, but I'm not seeing the removal of any -Wno-orphans
, which is supposed to be the main benefit of this. Do we not have any examples or tests that are affected?
{-# LANGUAGE DerivingVia #-} | ||
{-# LANGUAGE FlexibleInstances #-} | ||
{-# LANGUAGE TemplateHaskell #-} | ||
{-# LANGUAGE TypeApplications #-} | ||
|
||
{-# OPTIONS_GHC -Wno-orphans #-} |
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.
There are a bunch of these in plutus-ledger-api. What are they for?
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.
Not sure. Pretty likely they became obsolete. I've removed them, lets and if build doesn't fail (which it didn't locally).
1ebe4ce
to
bac2190
Compare
This PR contains changes in these categories:
AsDefinitionId
typeclass renamed toHasBlueprintDefinition
HasSchema
typeclass renamed toHasBlueprintSchema
Unroll
type-family is made associated with theHasBlueprintDefinition
typeclass, allowing extensibility.HasBlueprintSchema
andHasBlueprintDefinition
instances for data types in theplutus-ledger-api
and ones exported by thePlutusTx.Prelude
.Closes #6265