search

michaelwittig / node-q

Q interfacing with Node.js
MIT License
52 stars 14 forks source link

BigInt for long #48

Open emilbayes opened 4 years ago

emilbayes commented 4 years ago

Hi!

Can I choose a custom serializer/deserializer for data types? I'm specicially wanting longs to be represented as JS native BigInt type. I'm also happy to provide a PR if this is something you'd want in the module

michaelwittig commented 4 years ago

Hi!

We only have the long2number setting to represent longs as https://www.npmjs.com/package/long

Since we now have BigInt your suggestion makes a lot of sense to use them.

PR is welcome. If possible, the changes should be backward compatible.