refactor(kad): rename to follow naming convention across repository

[PanGan21]

Description

Renamed the following

kad::Kademlia -> kad::Behaviour
kad::KademliaEvent -> kad::Event
kad::KademliaBucketInserts -> kad::BucketInserts
kad::KademliaStoreInserts -> kad::StoreInserts
kad::KademliaConfig -> kad::Config
kad::KademliaCaching -> kad::Caching
kad::KademliaEvent -> kad::Event
kad::KademliaConnectionType -> kad::ConnectionType
KademliaHandler -> Handler
KademliaHandlerEvent -> HandlerEvent
KademliaProtocolConfig -> ProtocolConfig
KademliaHandlerIn -> HandlerIn
KademliaRequestId -> RequestId
KademliaHandlerQueryErr -> HandlerQueryErr

Resolves: #4485

Notes & open questions

Change checklist

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• A changelog entry has been made in the appropriate crates

[PanGan21]

I am wondering if non exported variable names from other files like KademliaHandler should also be renamed or it is just client faced change?

[thomaseizinger]

I am wondering if non exported variable names from other files like KademliaHandler should also be renamed or it is just client faced change?

We should rename the internal ones too but we don't have to add type-aliases for those :)

[thomaseizinger]

Have a look at the CI failures. I think the type-aliases will have to go to the top-level module to be publicly visible :)

[thomaseizinger]

This is great, thank you! To finish this off, can you please:

• Bump the patch version in protocols/kad/Cargo.toml
• Make the same bump in [workspace.dependencies] in Cargo.toml
• Write a changelog entry in protocols/kad/CHANGELOG.md

[PanGan21]

This is great, thank you! To finish this off, can you please:

• Bump the patch version in protocols/kad/Cargo.toml
• Make the same bump in [workspace.dependencies] in Cargo.toml
• Write a changelog entry in protocols/kad/CHANGELOG.md

Done! Thanks for the guidance!
I started reading/learning about Kademlia while doing this pr so I will look for related issues :)

[thomaseizinger]

Thanks!

Approvals have been dismissed because the PR was updated after the send-it label was applied.

[mergify]

This pull request has merge conflicts. Could you please resolve them @PanGan21? 🙏