-
Notifications
You must be signed in to change notification settings - Fork 3
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
[email protected]
compatibility
#1
Comments
Good idea. Could you install the deps using pip3 though? I'm so tired of node_modules bloat. Lets rewrite this in python and ship a WASM binary :DDD |
@paoloricciuti you should submit a PR with 100 chungus of dependencies then only it is acceptable |
bro, i got the solution, will just rewrite this in c, then use node bindings |
@paoloricciuti You'd need to introduce custom GitHub Actions to be able to run tests for that. And you better rewrite them in tape |
I don't have a heart to close this. Pinning it to the top 🤣 EDIT: It seems it stays pinned even after being closed so will close this, but good thread 😄 |
Lol |
This package doesn't seem to be compatible with
[email protected]
which is exactly what I needed. Is it possible to be compatible with it?I'm willing to make a PR for it but I will likely need to bring some dependencies. That could also help with security since currently this package is at risk of someone messing with the prototype of native objects.
The text was updated successfully, but these errors were encountered: