-
Notifications
You must be signed in to change notification settings - Fork 0
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
rollup / packaging issues #32
Comments
- Suppress a warning - Rename some output files - Remove redundant declaration files Address #32
This one is related to our use of Also related to this issue is removing the |
Can we get rid of Buffer? Just use string concatenation? bigint-mod-arith is a remarkably small package and we're only using a small part of it. We could just decide to copy that into our own code, so long as the license is compatible. |
Unfortunately doesn't fix the circular dependencies warning because we still indirectly depend on Buffer. Address #32
Temporary hack (TM) to get around an upstream packaging issue. Address #32
Right now, when you run
npm run test
, it proceeds to start running the packaging process. Dunno why that's happening, but something's not right.There are also warnings about circular dependencies and about
this
being rewritten toundefined
, so I'm assuming that we've still got some effort to get this all buttoned up.The text was updated successfully, but these errors were encountered: