-
-
Notifications
You must be signed in to change notification settings - Fork 637
Rename conept exercise dirs: closures
and recursion
#1237
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
Rename conept exercise dirs: closures
and recursion
#1237
Conversation
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.
We probably need to fix more in those exercises, so putting it on hold
In
Are you sure not merging this on its own is the best strategy? I'm not really volunteering to fix |
It's a good indication currently on if it's old or not :p. That's my only concern. I can probably make time this weekend. |
Narrator: I didn't |
@angelikatyborska I discussed with @SleeplessByte that I will have look whether anything else needs to be fixed. |
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.
Discussed with @SleeplessByte that we put the recursion exercise back to "wip" for now because too many things are missing.
Added some more small fixes and the missing blurbs.
I noticed that there still are some concept exercises that are in directories named after concepts.
This PR renames:
closures
->coordinate-transformation
recursion
->pizza-order
There are 3 non-story directory names remaining, but I am not touching them:
array-loops
because it's already done in this draft PR [V3] Elyses Looping Enchantments: fix everything around this exercise #1097errors
because it's not in the mainconfig.json
so it doesn't have a name. Should it be added there? is it waiting for improvements?nullability
because it's deprecated