-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add scala.quoted.Liftable
s to the stdlib
#6928
Add scala.quoted.Liftable
s to the stdlib
#6928
Conversation
quoted.Liftable[List[T]]
to the stdlibscala.quoted.Liftable
s to the stdlib
31c0f08
to
3340db1
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.
LGTM
given [T1: Type: Liftable] as Liftable[Tuple1[T1]] = new { | ||
def toExpr(tup: Tuple1[T1]) = | ||
'{ Tuple1(${tup._1.toExpr}) } | ||
} |
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.
Minor: maybe still write new Liftable[...]
to be consistent and explicit as above, the same for the code below
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.
It got way too long and made it harder to read.
08d9b6b
to
5796226
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.
LGTM
5d46185
to
36186f0
Compare
36186f0
to
81dd055
Compare
No description provided.