slog.Handler not implemented about weaver HOT 3 CLOSED

Hi @naivary, thanks so much for reporting this. It looks like our code depends on a stale version of the exp/slog module from 02/06/2023:

We'll work on updating the code to the latest version of exp.

from weaver.

My pleasure to contribute. I would like to contribute to the project but I have no idea where to start for the beginning to? I already read some main parts of the code and understood it (thanks for the nice written code and docs). Should I start with an issue first or try something else? Or try to update the package to use the newest version of slog?

from weaver.

Hi @naivary, we updated the code to the latest version of exp.

Contributions - Right now there are a few things you can help with:

1. Go over the issues, and see if they still make sense; In particular #106 might be useful to address.
2. Improving our testing coverage (just go through the code and try to add new tests where you see that they are missing/not enough)
3. While reading the code, if you find anything that can be improved (e.g., code cleanup, better code organization, we use deprecated APIs)
4. If you are more adventurous, you can read our blog post on how to write a new deployer and maybe you can try to write a simple deployer that can deploy a service weaver application in a VM, or you can improve our SSH deployer (few things in my mind: add health checks, add slow rollouts, add tests).

from weaver.