Adds lazy to implicit Encoder vals

[alycklama]

The implicit val encoderX ... need to become lazy, as the hierarchy matters:

case class Organization(owner: Person)
case class Person(name: String)

implicit val encoderOrganization: Encoder[Organization] = ...
implicit val encoderPerson: Encoder[Person] = ...

The above will throw a NullPointerException once you start encoding an Organization.

[ghostbuster91]

You are right. Probably the proper way to fix that would be to analyze dependencies between products and generate them in a way that takes that into account. Although, that would be a lot of work.

As a first implementation your solution is good enough 👍

[ghostbuster91]

Thank you for the contribution! 🎉

[alycklama]

You're welcome!

The dependencies could be analyzed, though I'm wonder what the added benefit is? The lazy keyword does this dependency resolution automatically for you. I don't believe there's any performance penalty to take into consideration here either for using lazy.

[ghostbuster91]

I like my generated code to be as efficient as possible even if the difference is negligible (e.g. lazy generates more bytecode). However, we should also take into account future maintenance and it is definitely easier to maintain a code that hasn't been written at all. So, fair point and I agree.

[ghostbuster91]

Published as 0.3.0 😄