horseradish shall be added #138
Conversation
|
what about #134 ? |
|
The implementation on this PR is better. |
|
I think we should close #134 then |
|
First of all, this PR lacks the essential horseradish image. This is required for any complete implementation of horseradish - you can see #134's approach here. More importantly, this PR seeks to deny horseradish of its instrument-hood, which is absolute blasphemy to the highest degree. This denial is an affront to Potato-kind and should be treated accordingly. I'm outraged that anyone could possibly consider this implementation to be even comparable to #134, let alone better! |
|
I completely agree with the indepth explanation that @md678685 gave about why this PR is low quality, we need to have some quality guidelines. The missing image makes me think this PR is just for some hacktoberfest points, but considering @ThatOneTqnk also created the pervious (#134) PR, i don't think this is the case now To be clear, its a high priority that we include images, else the support for horseradish is lacking and this might cause additional issues down the road. My suggestion would be however that we improve this PR, because #134 included some unwanted changes. |
|
Needs unit tested. -1 |
well hopefully it will