-
Notifications
You must be signed in to change notification settings - Fork 13
Forth test #950
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
Forth test #950
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
|
@SleeplessByte as discussed in the forum |
|
I believe the problem is in RegExp.escape |
You're expectation is correct. This is not supported before Node 24.
The current LTS (long term support) is 22, which is what we support officially. That also means that we would expect this to fail for you using the CLI and not the browser (as discussed on the forum). We can leave this PR open and merge it when LTS moves to 24, later this year:
|
|
Yeah, we use it when you run it locally! Check out https://github.com/exercism/babel-preset-javascript/blob/main/index.cjs#L11 which loads I thought we set-up this test-runner to use the same, but it seems to not do that? Imma try out to update the test-runner, and then we can merge in that update here and see if that makes it work. edit: yeah we do, but we were running 3.38 instead of 3.44 |
7bc634c to
8cd3023
Compare
Removing the personal metadata
|
More data:
We were running polyfill 3.38, but now that I bumped it to I generated |






Files for a failing test to the Forth exercise, with a results.json (renamed as expected_results.json) and the files for the exercise.