-
Notifications
You must be signed in to change notification settings - Fork 5
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 test runner injection patterns #277
Conversation
Codecov ReportBase: 94.22% // Head: 93.39% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #277 +/- ##
==========================================
- Coverage 94.22% 93.39% -0.83%
==========================================
Files 62 63 +1
Lines 5144 5180 +36
==========================================
- Hits 4847 4838 -9
- Misses 297 342 +45
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Fun weekend refactoring project ;-p
Haven't changed any functionality here, just some nice tidy-up I've been wanting to do forever. Main objectives:
There are some naming changes and reorganising the modules a little too.
I don't think there is anything controversial here, mostly involved a lot of brainless method renaming 🤣 so I don't think a thorough review is required. Happy to chat about the naming choices if you have input there though @adzialocha.
closes: #276 #208 #183
📋 Checklist
CHANGELOG.md