Replace deprecated usages of util.isDate & util.isString

[ramikg]

The functions util.isDate & util.isString have been deprecated for a few years, and have been removed in Node.js 23.

In order for nodejs-driver to be supported on recent Node.js versions, I've replaced their usages with the ones recommended in the docs.

[ruben-chavarria]

Yep, I'm having the same issue on Node.js v22

(node:66) [DEP0047] DeprecationWarning: The `util.isDate` API is deprecated.  Please use `arg instanceof Date` instead.
(Use `node --trace-deprecation ...` to show where the warning was created)

Using flag --trace-deprecation:

(node:17) [DEP0047] DeprecationWarning: The `util.isDate` API is deprecated.  Please use `arg instanceof Date` instead.
    at /app/node_modules/cassandra-driver/lib/utils.js:216:14
    at Array.forEach (<anonymous>)
    at Object.deepExtend (/app/node_modules/cassandra-driver/lib/utils.js:200:11)
    at Object.extend (/app/node_modules/cassandra-driver/lib/client-options.js:116:25)
    at new Client (/app/node_modules/cassandra-driver/lib/client.js:402:32)
...

This PR seems to be a good fix for the deprecation warning 👍

[ramikg]

This PR seems to be a good fix for the deprecation warning 👍

Thank you. Unfortunately, I'm not sure that this repository is being maintained.

[SiyaoIsTraveling]

Hi @ramikg, thank you for your contributions to this project! Yes this project is being maintained :)
I've created a ticket for this PR: NODEJS-682. Both this PR and your PR to remove @types/long dependency look good to me and I would love them to get merged.
Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

[ramikg]

Glad to hear that. Also, I signed the CLA.

[SiyaoIsTraveling]

Thanks! LGTM! util.isDate and util.isString are deprecated and this PR replaced them by the practices suggested by the official doc. Please merge as long as it passes all the tests @absurdfarce .

[absurdfarce]

Looks good to me as well; thanks for the PR @ramikg! Agreed that this seems quite sensible.

I've kicked off a Jenkins run for this change, so as long as we don't have any giant regressions or anything we should be fine to get this merged.

[absurdfarce]

Jenkins run was indeed green, calling this good

[ramikg]

Thank you!