-
Notifications
You must be signed in to change notification settings - Fork 837
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.
Looks fine overall. Please address comments before merging.
5ccd510
to
947ed60
Compare
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 convert the images the JPG - the PNG's with many colors and lots going on are very large to show inline in a webpage. (3 & 6 MB)
00e4aa3
to
e585896
Compare
Codecov Report
@@ Coverage Diff @@
## dev-preview4 #733 +/- ##
================================================
+ Coverage 86.9% 86.95% +0.04%
================================================
Files 125 125
Lines 4345 4345
Branches 827 827
================================================
+ Hits 3776 3778 +2
+ Misses 567 565 -2
Partials 2 2
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## dev-preview4 #733 +/- ##
============================================
Coverage 86.9% 86.9%
============================================
Files 125 125
Lines 4345 4345
Branches 827 827
============================================
Hits 3776 3776
Misses 567 567
Partials 2 2 Continue to review full report at Codecov.
|
* update Readme.md and Contributing.md * remove markdown-toc * new screenshots ( jpg ) * add mouse and keyboard shortcuts from vott-ct #AB17930
going to hold on this until we flip over v2 -> master