-
Notifications
You must be signed in to change notification settings - Fork 5
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
Small changes to index, .gitignore #14
Conversation
Codecov Report
@@ Coverage Diff @@
## master #14 +/- ##
=======================================
Coverage 98.26% 98.26%
=======================================
Files 7 7
Lines 231 231
=======================================
Hits 227 227
Misses 4 4 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.
These look good, should be fine to merge into master.
Can I do that?
…On Fri, Aug 3, 2018 at 11:06 AM, Alexander L. Hayes < ***@***.***> wrote:
***@***.**** approved this pull request.
These look good, should be fine to merge into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#14 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AWQ7T38iWrH1urYyuuQbLX2Lbe-C4YSuks5uNHUDgaJpZM4VuO4D>
.
|
@skinn009 Yep. You should have a "Merge pull request" option now that the changes are approved, if it isn't appearing let me know (I was tweaking some access options when adding you and Yuqiao, but if it isn't appearing I should be able to fix it). |
Edits to documentation. |
Okay- I think I merged. Thanks.
Mike
…On Fri, Aug 3, 2018 at 11:17 AM, Alexander L. Hayes < ***@***.***> wrote:
@skinn009 <https://github.com/skinn009> Yep. You should have a "Merge
pull request" option now that the changes are approved, if it isn't
appearing let me know (I was tweaking some access options when adding you
and Yuqiao, but if it isn't appearing I should be able to fix it).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AWQ7T4kg9a-aHrFGxZpKjA18DN3EyhiEks5uNHemgaJpZM4VuO4D>
.
|
Added pycharm environment control files to .gitignore.