Add save-the-cow, deprecating hangman#2652
Conversation
|
I'm good with the name change, but To be fair, FRP is a complex thing, I tried implementing |
|
I removed the mention thinking tracks could implement it without FRP and then not need an instructions append to clarify FRP was in fact not needed. However, I can put it back in since the exercise is intended to replace Hangman, and the forum discussion approving the PR to be made didn’t touch upon the content of the instructions. |
|
Would it make sense for exercises to come with optional appends that tracks could add that would include suggestions about intents/approaches/concepts? Multiple tracks implement hangman without FRP. But it would be nice not to drop it completely. |
|
That would mean the backend code would need to take those extra files into account, and I don't see Jeremy having time to do that anytime soon. Maybe instead of having the mention of FRP in the instructions, we put it in the top level comments in the canonical data? |
I was suggesting additional md files in the problem spec that maintainers could opt to rename to |
|
Ah right. Yeah that is an option |
Would it be appropriate for a comment to be inserted in the canonical data JSON to suggest a FRP approach if applicable? Then it's not dropped completely and the maintainers can choose if they want to use FRP or not. |
I think it's always safe to dump comments into the canonical data file :) I think those are easier to miss and aren't markdown so it might not be quite as nice, but that works for me. |
mk-mxp
left a comment
There was a problem hiding this comment.
I'd also add the FRP bit as a comment to the JSON.
|
Looks good to me |
|
Realistically, the added comment won't make any difference, I'm guessing after reading it, hardly any track will go out of their way to implement the exercise with FRP. |
I'm proposing to deprecate Hangman due to its violent subject matter and replace it with a gentler Save the Cow exercise.
See http://forum.exercism.org/t/replace-hangman-with-non-violent-content/34156.