-
-
Notifications
You must be signed in to change notification settings - Fork 72
[elyses-enchantments] Concept exercise #833
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
Conversation
Working towards getting a root and a trunk for the start of the syllabus, I think this is where it's at:
Of course, there is also a lot of stuff lower down that's making good progress. However, getting to see the start of a coherent syllabus on the Exercism website is becoming a priority for me (you may hear that old people learn patience, but only up to a point). |
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.
There appears to be some discrepancy with the naming of some files. Should it be "enchantmants" or "enchantments"? I'm not sure if this was done with some purpose to avoid the old name or not.
Otherwise, there are a couple of small typos in design.md
, introduction.md
and elyses-enchantmants.jl
, which I'll just edit directly.
Changes typo "enties" to "entries"
Add missing "of"
Add new line to function definitions
That was 100% unintended screwup, where my fingers bypassed my brain. It should always be "enchantments". |
In case you wondered how I managed to get it wrong in so many different places: I used a Bash script to generate the exercise structure, so once I made a typo on the command line it propagated widely. Also, it's been a bad few days for |
We got there in the end... |
I think this is the current top-of-syllabus progress:
That gives us 3 boxes on Exercism, with 3 more well advanced. I'm partway through implementing I also found my 80%-complete draft of the |
Looking good! I'll try to keep reviewing, so we can keep moving forward. |
Pairs with the
vectors
concept.Content is mostly new, ported from what I previously did for R. The earlier draft in
exercises/concept.wip
is removed to avoid conflicts.