search

amazon-ion / ion-js

A JavaScript implementation of Amazon Ion.
http://amzn.github.io/ion-docs/
Apache License 2.0
352 stars 48 forks source link

Add a BigInt build target #477

Open zslayton opened 4 years ago

zslayton commented 4 years ago

Now that we've migrated to JSBI, we should make it possible for users to build a version of ion-js that uses BigInts instead of JSBI values.

omichowdhury commented 2 years ago

Hi is this being worked on?

zslayton commented 2 years ago

Hi @omichowdhury, this is not in active development.

If we produce a BigInt build target, it will use the same source code but will produce a second version of the library that uses BigInt. Currently, ion-js's test suite is in TypeScript. We would probably need to compile the test suite to JavaScript to be able to run it against both build targets' output (JS using JSBI and JS using BigInt).

BigInt is now pervasive enough that we may be able to consider a major version bump that migrates from JSBI to BigInt. Can you tell me about your platform and use case?

omichowdhury commented 2 years ago

Hey @zslayton

We're using QLDB in a typescript/lambda (node v14) environment. We use SST (framework on top of CDK), which uses esbuild for transpilation & bundling, and use @swc/jest for transpiling and running our unit & integration tests.

Happy to jump on a call if you want more details