-
Notifications
You must be signed in to change notification settings - Fork 16
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
Bug in borer-derivation? deriveCodec fails to compile when using type alias #388
Comments
Ok, thank you for reporting. |
Thank you, @MarconZet, for reporting. |
We'd appreciate a patch release :) |
Ok, just pushed borer 1.7.1 to sonatype. |
Wow that's one of the fastest report-to-release cycles I've seen in OSS so far ❤️ |
:) |
Actually, FYI, as you've probably noticed, we're working on ready-to-use serializer for Akka messages/event/states, since Jackson that's suggested by Akka docs is a joke (surprising runtime failures, no support for Scala Are you aware of any work that has been already done in that direction? There's borer-akka-compat but AFAICS it doesn't provide a ready Serializer, just Codecs for ActorRefs etc. |
Also @sirthias ... would it be possible to retain support for Scala 2.12 in 1.7+? Since AFAICS 1.7.0 and 1.7.1 have only been published for 2.13 :// |
Yes. The reason is that we haven't needed these things ourselves yet. |
Great! We're prototyping that as a separate project for convenience as for now, but it makes sense to integrate it back into main borer repo... opened VirtusLab/akka-serialization-helper#18 for that |
@PawelLipski We are using borer for akka-serialization using the pattern mentioned here. |
@mushtaq actually! since you're using Borer for a bit longer than us... do you have any experiences with migrating Akka events when Borer is used as a serializer (or any other library, for that matter)? |
Unfortunately, no. So far, we do not have to deal persisting these events. I just came across transducers in latest version of borer. Will that help? |
Looks promising! Let us take a look 👁️ |
fails build with:
Expected behaviour: Build successful
I found a workaround with:
but that's far from an elegant solution.
The text was updated successfully, but these errors were encountered: