-
Notifications
You must be signed in to change notification settings - Fork 60
Added documentation for some functions and methods. #123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Thank you for working on this one! I'd suggest: i) fixing some typos regarding the proper use of Rust's docstrings (i.e.: missing whitespace on |
fixed typos and added documentation to more public functions, methods, structs and traits I could find. |
Pull Request Test Coverage Report for Build 14520630096Details
💛 - Coveralls |
You can squash the changes into a single commit, and use the conventional commits terminology, e.g Also, there are still some formatting issues, see the CI output, you can run |
e436e40
to
6b2af4c
Compare
made the required changes, can you review it once. |
@psg-19 your documentation should follow the recommended format from the docs here: In particular start with a concise summary. For these that's all that may be needed. But if more details are needed put those on a following paragraph or lines. It's OK to borrow wording from other projects like |
I've submitted a PR to address this issue: #109
Tried documenting some of functions and methods.