-
Notifications
You must be signed in to change notification settings - Fork 93
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
Included limitations #211
base: master
Are you sure you want to change the base?
Included limitations #211
Conversation
Spent 5 hours trying to mock standalone functions using ts.
Codecov Report
@@ Coverage Diff @@
## master #211 +/- ##
==========================================
+ Coverage 93.55% 94.23% +0.67%
==========================================
Files 34 34
Lines 652 659 +7
Branches 80 80
==========================================
+ Hits 610 621 +11
- Misses 30 33 +3
+ Partials 12 5 -7
Continue to review full report at Codecov.
|
|
@NagRock Its been more than a year since this small readme change was proposed, any reason as to why this shouldn't be merged? 🤔 |
Looks like the review feedback from a year ago was only addressed 10 minutes ago. Also might wanna take a look at #183 |
@cspotcode this library has ~150,000 weekly downloads. Sad to see it not getting the attention it deserves. Maybe the @mockitoguy should adopt this. |
Check out relevant discussion in #183. Lots of community voices appeared to be opposed to realistic forms of adoption. |
Spent 5 hours trying to mock standalone functions using ts.