-
Notifications
You must be signed in to change notification settings - Fork 291
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
.workplane docstring update #563
Conversation
@adam-urbanczyk hello, please also check the quick start manual, section "More Holes", line 10, point 2 (it says that "The center point of a workplane on a face is always at the center of the face, which works well here") |
Codecov Report
@@ Coverage Diff @@
## master #563 +/- ##
=======================================
Coverage 94.19% 94.19%
=======================================
Files 29 29
Lines 6220 6220
Branches 665 665
=======================================
Hits 5859 5859
Misses 224 224
Partials 137 137
Continue to review full report at Codecov.
|
Looks good to me! |
I'm re-running two CI checks that seem to have failed for no reason. |
Last two CI checks pass now after being run again. Has the comment from @adam-trhon been addressed? I'm not sure what the intent of that comment was. |
@jmwright If I understand the quoted sentence right, it refers to the previous defualt (which was |
@adam-trhon @jmwright I removed the confusing line completely. Do you think that the tutorial remains clear? |
+1 |
Alright, merging. |
No description provided.