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

[refined4s-circe] Update refined4s.modules.circe.derivation.instances.derivedEncoder to use contraCoercible and make it inline #154

Closed
kevin-lee opened this issue Dec 23, 2023 · 0 comments · Fixed by #155
Assignees
Labels
Milestone

Comments

@kevin-lee
Copy link
Owner

Summary

[refined4s-circe] Update refined4s.modules.circe.derivation.instances.derivedEncoder to use contraCoercible and make it inline

Version

0.4.0

Description

The current code has the following issue if it's turned into an inline method.

[error] 13 |    a => encoder(coercible(a))
[error]    |    ^^^^^^^^^^^^^^^^^^^^^^^^^^
[error]    |An inline given alias with a function value as right-hand side can significantly increase
[error]    |generated code size. You should either drop the `inline` or rewrite the given with an
[error]    |explicit `apply` method.
@kevin-lee kevin-lee added the task label Dec 23, 2023
@kevin-lee kevin-lee added this to the m5 milestone Dec 23, 2023
@kevin-lee kevin-lee self-assigned this Dec 23, 2023
kevin-lee added a commit that referenced this issue Dec 23, 2023
…ion.instances.derivedEncoder to use contraCoercible and make it inline
kevin-lee added a commit that referenced this issue Dec 23, 2023
…ion.instances.derivedEncoder to use contraCoercible and make it inline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment