-
-
Notifications
You must be signed in to change notification settings - Fork 159
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 script to scaffold new practice exercises #716
Conversation
- The tests should be based on the canonical data at 'https://github.com/exercism/problem-specifications/blob/main/exercises/${slug}/canonical-data.json' | ||
- Any test cases you don't implement, mark them in 'exercises/practice/${slug}/.meta/tests.toml' with "include = false" | ||
- Create the example solution in $(jq -r '.example' <<< "${files}") | ||
- Verify the example solution passes the tests by running 'bin/verify-exercises ${slug}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This refers to a bin/verify-exercises script that appears to be missing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah. I'll update! Well spotted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #717
6a9e399
to
a506a33
Compare
I've also updated the PR to write the namespaces to the solution, example and test files. |
So is the scaffolding supposed to run only on linux? I tried it on windows with 'git bash' and it works fine. Only verifying the exercise seems to fail. |
For Windows, I would recommend running it on WSL. Most tracks are maintained by users that are on Linux, or else have access to WSL. |
Thanks, i'll give WSL a go. This will take some time given it's the first time i have to look at anything but exercises. So I'd like to do this properly, but if you are in a hurry, please go ahead and merge. |
No description provided.