-
Notifications
You must be signed in to change notification settings - Fork 1
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
Various additions #1
Conversation
Instead of defining derivation via a given, make it explicit via `macroR` and `macroW` defs. This also copies upickle's original behaviour. Also remove wildcard imports from the compiletime and deriving packages.
def serializeDefaults: Boolean = false | ||
|
||
def objectAttributeKeyReadMap(s: CharSequence): CharSequence = s | ||
def objectAttributeKeyWriteMap(s: CharSequence): CharSequence = s | ||
|
||
def objectTypeKeyReadMap(s: CharSequence): CharSequence = s | ||
def objectTypeKeyWriteMap(s: CharSequence): CharSequence = s |
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 stuff was moved from Api
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.
Otherwise LGTM. Are the Dotty and Scala 2 tests passing locally?
end getDefaultParmasImpl | ||
|
||
inline def summonAll[T <: Tuple]: List[_] = | ||
inline def summonList[T <: Tuple]: List[_] = |
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.
summonAll was defined here temporarily while we didn't have one in Dotty. Now we do. Can we completely remove summonAll and summonList from here and use the Dotty implementation?
Currently WIP, these changes aim to bring the upickle community build up to feature parity with scala 2.
Some notable changes:
macroR
,macroW
andmacroRW
upickle.implicits.key
annotationMany tests still fail and/or need to be reactivated