-
Notifications
You must be signed in to change notification settings - Fork 78
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
CLIENT-2749 #599
CLIENT-2749 #599
Conversation
Nodejs-Release-5.9.0
CLIENT-2672 CLIENT-2709 CLIENT-2749 CLIENT-2750 Changed npm-run-all from a dev-dependency to a dependency. Added missing BOOL enumeration to Aerospike.exp.type. In scan/query with maxRecords set, mark node's partitions for retry on next scan/query page when that node returns records that are discarded due to exceeding maxRecords.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stage #599 +/- ##
==========================================
+ Coverage 91.55% 91.68% +0.13%
==========================================
Files 69 69
Lines 2250 2250
==========================================
+ Hits 2060 2063 +3
+ Misses 190 187 -3 ☔ View full report in Codecov by Sentry. |
CLIENT-2751 Added support for Node.js 21.
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.
FWIW LGTM, other than one suggested change.
Added node 21 to testing matrix
…an openssl version
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.
FWIW, LGTM
No description provided.