search

Rantanen / node-mumble

Mumble client in Node.js
MIT License
155 stars 48 forks source link

Rename MumbleWrapper to MumbleClient maybe? #31

Closed Rantanen closed 9 years ago

Rantanen commented 9 years ago

While MumbleWrapper makes sense from the internal point of view, now that the name is exposed to the users through documentation, etc. I think we should rename it to MumbleClient as wrapper might end up confusing the users?

This shouldn't affect the interfaces in any way so it's not as big of a change as it sounds like.

Thoughts?

Prior99 commented 9 years ago

+1

Rantanen commented 9 years ago

That's enough for me. Done. :)