-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Error in spec for palindrome-products #177
Comments
I can only infer from the outside what happened, so these are merely my assumptions: during deployment, the network connection became unstable, leading to the core package not being available. Since Array.prototype.push is the first polyfill to be called by Babel, the missing package leads to errors before the tests even run. Proposed solution: redeployment with new version. |
I have changed the spec not to use Array.prototype.push, but I could only locally reproduce the issue by removing the file that was missing in the testing back end. |
The spec change has successfully fixed I think the error was a consequence of Minimizing the test runner image size is important to save Exercism storage costs on the test runner servers. If we add to package.json or pnpm-lock.yaml, I'm not sure how we measure the increase in image size. If we add a ci workflow that calls wasm-test-runner, the problem won't reach users again. |
If we wanted to minimize the bundle size for the test runner, we should consider switching to vitest. On the other hand, we could consider moving away from JS and run the code using wasmer.io |
Described in
https://forum.exercism.org/t/error-in-spec-for-palindrome-products/16214
Reproduced:
Submitting the example solution gives
Possible steps for investigation:
The text was updated successfully, but these errors were encountered: