-
Notifications
You must be signed in to change notification settings - Fork 86
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
git workflow added #26
Conversation
Current coverage is 100% (diff: 100%)@@ master #26 diff @@
===================================
Files 5 5
Lines 162 162
Methods 0 0
Messages 0 0
Branches 0 0
===================================
Hits 162 162
Misses 0 0
Partials 0 0
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review. There are some typos and references to scikit-learn.
@vibrantabhi19, keep up the good work. While the GitHub workflow is well explained, it doesn't document a) How to contribute - improve documentation/add new methods/improve performance. Please let me know if you need any help or information. Also it might be better to document the overview of code structure and API can be written in a separate document rather than this. You can leave that section at this time. Thank you for your contribution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are still some typos, please proof-read the text. Thanks.
@vibrantabhi19, can you please update on this? Thanks. |
Sure. Will do it within a day. |
I have added the following points mentioned in Issue #14
a) How to contribute - improve documentation/add new methods/improve performance.
b) Explain contribution workflow - issues/pull-request/review.
Have a look and then I will proceed with the next.