-
Notifications
You must be signed in to change notification settings - Fork 11
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
add NJS support #71
Comments
TIL it uses its own engine (mostly?), definitely interested in adding it! |
Do they have or are willing to add the functions required for an eshost-compatible runtime? (https://github.com/CanadaHonk/eshost/tree/master/runtimes) |
Yep, completely from scratch as far as I'm aware.
I'll ask! |
I added basic NJS support to our eshost fork in CanadaHonk/eshost#17, the stubbed methods are what would be needed on their end for a complete runtime. This is not blocking, it just means that certain tests won't run. |
Draft PR for NJS test262.fyi integration: #72 |
@michaelficarra results are in: https://test262.fyi/#%7Cnjs |
See https://github.com/nginx/njs
NJS is a complete JavaScript engine developed to be embedded in nginx as a dynamic nginx module. NJS can be run outside of nginx using the CLI or the REPL: https://nginx.org/en/docs/njs/cli.html. It would be nice to be able to see how it compares to other popular engines on spec compliance. If you need help, I should be able to connect you with the NJS devs, so just let me know.
The text was updated successfully, but these errors were encountered: