-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
Update exercise generator tooling #382
Comments
After I removed the sed line, I received a subsequent error while compiling
I was able to remedy this by updating the dependencies inside the exercise_generator's gleam.toml file to their latest versions. Then the script ran through the end and generate the files I expected. |
The exercise generator generates Gleam files that don't pass the |
average_run_time
an integer in ./bin/generate_practice_exercise
I was the one who made the generator and added that line about the average run_time. The required format used to be the opposite of what it is now, it used to require a float, and it was so annoying :D Also, calling the generator from It's more than probable that the dependencies need to be updated though. |
Thanks, I'll take another look since that script didn't seem to format the files when I ported |
It was, which is why I changed it :D |
The offending line is
sed -i 's/"average_run_time": \([[:digit:]]\+\)$/"average_run_time": \1.0/' config.json.tmp
. Per #306,average_run_time
should be an integer in the first place so we don't need to work around jq.The text was updated successfully, but these errors were encountered: