zulip/python-zulip-api

Rename BotHandler to AbstractBotHandler.

rht opened this issue · 1 comments

rht commented

To prevent confusion about its usage. To cement the fact that it is an abstract class used only for typing annotation purpose.
See discussion at https://chat.zulip.org/#narrow/stream/127-integrations/topic/deploying.20bots.20on.20heroku.3F.3F.3F.20--.20BotHandler.

Still need discussion: whether we should additionally use NotImplementedError instead of ... to proactively tell the library users if they accidentally use the class.

I think a naming change is likely sufficient to avoid library users accidentally using this class; we can always do something further after if we see the same confusion after a rename.