Add simple-cipher exercise
#772
Conversation
ErikSchierboom
force-pushed
the
simple-cipher
branch
3 times, most recently
from
0b92183 to
e861317
Compare
There was a problem hiding this comment.
Hopefully, i did not miss anything.
We should probably use something more distinct than a simple dash to separate parent and child descriptions. If descriptions also have dashes, it'll make the full description difficult to parse. How about using >> (or the more distict ▶ )
| "uuid": "ce982624-94ac-4bcd-8946-0a20f99545d7", | ||
| "practices": [], | ||
| "prerequisites": [ | ||
| "conditionals", |
There was a problem hiding this comment.
Can you explain a bit the reasoning behind the conditionals?
There was a problem hiding this comment.
We can probably remove that. Originally I had used conditionals, but they're not needed.
There was a problem hiding this comment.
Your solution does not use them, but i'm wondering if conditionals are useful in case someone does not come up with a similar solution. I haven't solved the exercise, so i'm not sure. Your call.
There was a problem hiding this comment.
It's probably not needed
|
@tasxatzial I've created a PR to update the separator: #779 Let's get that merged first and then I'll update this Pr and the |
ErikSchierboom
force-pushed
the
simple-cipher
branch
from
ac464ca to
f7fea2b
Compare
Co-authored-by: Anastasios Chatzialexiou <[email protected]>
Co-authored-by: Anastasios Chatzialexiou <[email protected]>
Co-authored-by: Anastasios Chatzialexiou <[email protected]>
Co-authored-by: Anastasios Chatzialexiou <[email protected]>
ErikSchierboom
force-pushed
the
simple-cipher
branch
from
f7fea2b to
64d495c
Compare
No description provided.