-
Notifications
You must be signed in to change notification settings - Fork 168
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
refactor: Refactor MaterialInteractor
#2691
refactor: Refactor MaterialInteractor
#2691
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #2691 +/- ##
==========================================
- Coverage 49.56% 49.56% -0.01%
==========================================
Files 474 474
Lines 26928 26930 +2
Branches 12422 12423 +1
==========================================
Hits 13348 13348
+ Misses 4751 4748 -3
- Partials 8829 8834 +5 ☔ View full report in Codecov by Sentry. |
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.
Nothing substantial from my side. Seems like a good idea.
This PR currently has a merge conflict. Please resolve this and then re-add the |
Aimed to improve consistency and split surface and volume interactions into two logical parts
pulled out of #2603