search

peterolson / BigInteger.js

An arbitrary length integer library for Javascript
The Unlicense
1.12k stars 187 forks source link

add missing export statement #226

Closed pmcelhaney closed 1 year ago

pmcelhaney commented 2 years ago

fixes #225

peterolson commented 1 year ago

I'm worried that this might be a breaking change, since it currently uses a CommonJS export

pmcelhaney commented 1 year ago

I don’t remember what I was thinking when I did this. I might have tested by importing with ESM without checking that it still works with CJS. Judging from the branch name, I might not have tested at all.

You’re probably right. If it’s imported with CJS the default keyword will be a syntax error.