Proposal: Change `Core` to `ClientManager`
Opened this issue · 1 comments
sharafian commented
The name Core isn't very descriptive of what the class does. Because it serves mostly the same purpose as Client but manages the same operations on a list of plugins, it would be easier for people to understand and use if it were called ClientManager or something similar.
emschwartz commented
I'm still confused about what the value add for Core/ClientManager is. Right now it seems like it's mostly a map of Clients indexed by their address. It seems like there's scope for this to do more though, especially regarding routing, retries, etc.