-
-
Notifications
You must be signed in to change notification settings - Fork 192
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
added two tests for deep setRight and setLeft #1121
base: main
Are you sure you want to change the base?
Conversation
I suspect a patch to problem-specifications/zipper would be welcome as well. |
@MatthijsBlom sorry, what do you mean? is there anything else i should do for this pr? someone needs to kick the ci for me... |
What it means is to submit a pull request to https://github.com/exercism/problem-specifications/blob/main/exercises/zipper/canonical-data.json |
To elaborate: the tests are generated/derived from the problem specifications in the problem-specifications repo. This is done to make it easier to collectively maintain exercises among the many individual language tracks. I have repeatedly seen it suggested to send PR's to problem-specifications first/instead, hence my suggestion to approach them. |
@MatthijsBlom @petertseng how do i make the pr? ie, i can make a pr directly to that file, but it would seem at least that i need to supply a uuid? how do i derive that value? |
I really don't know, but I'm sure the maintainers of the problem-specifications repo will tell you. |
It is your choice how you generate the UUID, as long as it is a version 4 UUID. |
Oh, silly me, of course it doesn't matter how it is generated: it is unique, after all! |
No description provided.