fix: some of clippy pedantic warnings

[varex83]

This PR fixes most of the clippy::pedantic warnings:

cargo clippy --workspace --all-targets --all-features --exclude pluto-eth2api --fix  -- -W clippy::pedantic -A clippy::missing_errors_doc

also, replace some of the builders with bon

[emlautarom1]

Is this PR intentionally targeting your bohdan/add-connection-logger branch?

[emlautarom1]

LGTM.

I don't think we want to generally enable pedantic in the codebase since some suggestions end up being quite annoying (ex. prefer method reference over method call), but instead we should evaluate them individually (ex. always using underscores for large numbers sounds reasonable enough).

[emlautarom1]

I don't think we want to go the route of adding this docs into all Result functions: it will get annoying quite quickly and provides little actual value.

Same for panics.

[emlautarom1]

Does this PR make sense with the changes to #202 ?

[varex83]

Does this PR make sense with the changes to #202 ?

No, I'm closing it since creating a new one will be easier than fixing merge conflicts.