-
Notifications
You must be signed in to change notification settings - Fork 49
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
Fix the '1 face' mislabelling of selection for sweep segments #5183
Fix the '1 face' mislabelling of selection for sweep segments #5183
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
QA Wolf here! As you write new code it's important that your test coverage is keeping up. |
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.
Yeah I think this is the fix. The coercion of segment
to face
was from before we had artifact graph, so we had to pass through the segment that made the face to select a face.
…ing-of-selection-for-sweep-segments
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5183 +/- ##
========================================
Coverage 85.94% 85.95%
========================================
Files 90 90
Lines 32718 32893 +175
========================================
+ Hits 28121 28272 +151
- Misses 4597 4621 +24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…ing-of-selection-for-sweep-segments
…ing-of-selection-for-sweep-segments
…ing-of-selection-for-sweep-segments
* Make point-and-click Sweep generally available Fixes #5156 * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Trigger CI * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Trigger CI * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Trigger CI * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Remove replace /segment/face * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Selections change will be done in separate PR #5183 * Toolbar button * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Reset snaps * Revert screenshot --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Fixes #5182