-
Notifications
You must be signed in to change notification settings - Fork 20
test: adding test coverage for designer, sketch, misc #2070
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2070 +/- ##
==========================================
+ Coverage 92.61% 92.84% +0.23%
==========================================
Files 136 136
Lines 9812 9812
==========================================
+ Hits 9087 9110 +23
+ Misses 725 702 -23 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Few comments - thanks for the tests @syscordan!
…b.com/ansys/pyansys-geometry into test/syscordan_designer_misc_coverage
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.
Some last comments @syscordan - just for separation of concerns. Thanks!
…b.com/ansys/pyansys-geometry into test/syscordan_designer_misc_coverage
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!
Description
adding test coverage for designer, sketch, misc*
Checklist
feat: extrude circle to cylinder
)