-
Notifications
You must be signed in to change notification settings - Fork 299
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
Slight improvement of the sketch tutorial #921
Conversation
@Jojain what do you think? Is it clear(er)? |
Codecov Report
@@ Coverage Diff @@
## master #921 +/- ##
=======================================
Coverage 96.11% 96.11%
=======================================
Files 39 39
Lines 9130 9130
Branches 1017 1017
=======================================
Hits 8775 8775
Misses 206 206
Partials 149 149
Continue to review full report at Codecov.
|
Thanks @adam-urbanczyk this is much more clear now ! |
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.
Thanks for doing this @adam-urbanczyk
I made one extremely minor suggestion. It sounds like @Jojain approves, so can we merge?
Co-authored-by: Jeremy Wright <[email protected]>
I think this looks good. Can I merge it @adam-urbanczyk ? |
+1 |
Mention modes explicitly