-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Jint 3.1.0 #15730
Jint 3.1.0 #15730
Conversation
Just to be sure: did you test it? :) |
We have unit tests for the JavaScriptEngine. That one at least has some. |
Seems I will take the responsibility to add a functional test for almost the things in the framework :) |
Or let's start asking functional tests or unit test with each PR's if they are missing. I think that's where we are at. |
I'll follow up on automated QA, because we absolutely need to do more on that front. However, for now, as a PR author everyone should make sure that their code doesn't break anything (I mean, I hope this is not controversial). |
There was an error during the build :
|
I merged main on this branch and it fixed it. |
3.1 broke some advanced API signatures (preparing AST method's input and output types) so if the code still compiles, it should work as before (at least Jint's +60 000 tests still pass 😉). |
Thanks, @lahma! So, Antoine, did you test it? |
I tested with TheBlogTheme recipe, add a layer with a Javascript condition. |
Honestly folks, this release ain't that evil 😆 We incremented the version to warn about the possible compilation time breaking change. It's probably basically 3.0.2 -> 3.0.3 for you. |
https://github.com/sebastienros/jint/releases/tag/v3.1.0
https://www.nuget.org/packages/Jint/3.1.0