-
Notifications
You must be signed in to change notification settings - Fork 83
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
Added contribute to EvalML instructions #213
Conversation
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.
i think this is a good start. let's have @angela97lin review before merging when she's back. then we can test run it with dylan next week
contributing.md
Outdated
* install in edit mode with: | ||
```bash | ||
cd evalml # move to directory | ||
pip install -e . # install in edit mode |
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.
i think we have make installdeps
which can be used for development installation
Codecov Report
@@ Coverage Diff @@
## master #213 +/- ##
=======================================
Coverage 96.77% 96.77%
=======================================
Files 91 91
Lines 2354 2354
=======================================
Hits 2278 2278
Misses 76 76 Continue to review full report at Codecov.
|
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.
Made some minor comments but otherwise LGTM! Would definitely be great to have Dylan test-run and make comments too :)
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.
LGTM! :)
Added contributing instructions.