-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
chore: Remove obsolete actor #27147
chore: Remove obsolete actor #27147
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #27147 +/- ##
==========================================
+ Coverage 67.14% 69.47% +2.32%
==========================================
Files 1901 1901
Lines 74444 74444
Branches 8304 8305 +1
==========================================
+ Hits 49989 51721 +1732
+ Misses 22403 20670 -1733
- Partials 2052 2053 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ 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.
LGTM
SUMMARY
Previously, in #20499, I removed the unnecessary and now obsolete "actor" construct. It seems like I missed one instance.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
CI.
ADDITIONAL INFORMATION