Add docstrings to the file.

Question

Add docstrings to the file.

Opened this issue 7 years ago · 3 comments

The file chat_server.py needs docstrings.

Answer 1 · 2017-10-30T13:27:57.000Z

Hi Hamer, Sure you can work on it. This is open source and any one can send the PR.

On Mon, Oct 30, 2017 at 6:29 PM, Hamer Iboshi ***@***.***> wrote: I will work on it. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AZZHvK5JnoQOkqad85VzWHmcfeNgSnwwks5sxchLgaJpZM4QLA37> .

Answer 2 · 2017-10-30T16:24:02.000Z

Hey,

There seems to be some docstrings already. Should all the methods have docstrings?

Answer 3 · 2017-10-30T16:50:11.000Z

Yes, all the methods should contain the docstrings. Moreover, add what the method takes as an argument, what it returns and what exceptions(if any) it handles.

…

On Mon, Oct 30, 2017 at 9:54 PM, Vesa Kivistö ***@***.***> wrote: Hey, There seems to be some docstrings already. Should all the methods have docstrings? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AZZHvEVICO8zBLCykh0MKfl_Xcm7KX5Nks5sxfgjgaJpZM4QLA37> .