Skip to content

Add comments to make the working of the functions clear.#1

Open
pratap-is-here wants to merge 2 commits into
jedijulia:masterfrom
pratap-is-here:patch-1
Open

Add comments to make the working of the functions clear.#1
pratap-is-here wants to merge 2 commits into
jedijulia:masterfrom
pratap-is-here:patch-1

Conversation

@pratap-is-here

Copy link
Copy Markdown

I added comments for every function to make it's working clear and I also added information about the rule a function implements.

I added comments for every function to make it's working clear and also information about the rule which a function implements.
@mfigurski80

mfigurski80 commented Jul 18, 2020

Copy link
Copy Markdown

I know this library may no longer be maintained but this could be useful for future reference. I suppose it should be converted to docstring though

@pratap-is-here

Copy link
Copy Markdown
Author

Thanks for the suggestion. I changed the comments to doc string. Let me know if it's better now.

@mfigurski80

Copy link
Copy Markdown

That's actually pretty awesome, thanks for doing that.

@jedijulia, it would be great if this could be merged. What do you thing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants