kubewarden / kubewarden-controller

Manage admission policies in your Kubernetes cluster with ease
https://kubewarden.io
Apache License 2.0
191 stars 33 forks source link

Epic(Go): move away from easyjson #492

Closed flavio closed 1 year ago

flavio commented 1 year ago

We have been using easyjson as a way to workaround TinyGo not supporting the reflection module. However, starting from the 0.28 release, the reflection capabilities have been added to TinyGo. Because of that, we can stop using easyjson.

Why we are doing that

Using easyjson requires policy developers to learn one more tool. By using vanilla encoding/json we will improve the developer experience.

Caveats

Easyjson main purpose is provide fast serialization/deserialization. However, looking at some work we've already done, the performance of policies are not going to be significantly impacted by this transition.

Checklist

flavio commented 1 year ago

All the tasks have been done, closing.

Thanks to all the ones involved! 👏