-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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: Replace cursor.count
with calls to countDocuments
or estimatedDocumentCount
#33725
Conversation
Looks like this PR is ready to merge! 🎉 |
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #33725 +/- ##
========================================
Coverage 75.55% 75.55%
========================================
Files 492 492
Lines 21558 21558
Branches 5373 5373
========================================
Hits 16288 16288
Misses 4630 4630
Partials 640 640
Flags with carried forward coverage won't be shown. Click here to find out more. |
44deaa9
to
b126b37
Compare
…matedDocumentCount` (#33725)
3/3
Proposed changes (including videos or screenshots)
Issue(s)
https://rocketchat.atlassian.net/browse/CORE-763
Steps to test or reproduce
Further comments