-
Notifications
You must be signed in to change notification settings - Fork 68
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
Dynamic smithy4s enums + allow IntEnum hint in JSON codecs #406
Dynamic smithy4s enums + allow IntEnum hint in JSON codecs #406
Conversation
modules/codegen-plugin/src/sbt-test/codegen-plugin/defaults/src/main/scala/Main.scala
Outdated
Show resolved
Hide resolved
modules/codegen-plugin/src/sbt-test/codegen-plugin/defaults/src/main/scala/Main.scala
Show resolved
Hide resolved
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Outdated
Show resolved
Hide resolved
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Outdated
Show resolved
Hide resolved
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Outdated
Show resolved
Hide resolved
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Outdated
Show resolved
Hide resolved
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Outdated
Show resolved
Hide resolved
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Outdated
Show resolved
Hide resolved
@@ -389,7 +484,7 @@ private[dynamic] object Compiler { | |||
) | |||
} | |||
val memberHints = allHints(mShape.traits) | |||
lField.map(_.addHints(memberHints: _*)) | |||
lField.map(_.addHints(memberHints.all.toSeq: _*)) |
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 is one problematic case - I wanted to add an addHints
overload to SchemaFieldOps
, but that breaks binary compatibility on 2.12 (the existing extension is renamed due to weird 2.12 compiler behavior). We can consider it for 0.16.0, in the meantime here's the workaround.
modules/dynamic/src/smithy4s/dynamic/internals/DynamicModelCompiler.scala
Show resolved
Hide resolved
Added a test in ae28247 |
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.
Let's gooooo
No description provided.