Potato icon indicating copy to clipboard operation
Potato copied to clipboard

horseradish shall be added

Open chatasma opened this issue 6 years ago • 6 comments

well hopefully it will

chatasma avatar Oct 01 '19 21:10 chatasma

what about #134 ?

Xeyame avatar Oct 01 '19 21:10 Xeyame

The implementation on this PR is better.

drtshock avatar Oct 01 '19 21:10 drtshock

I think we should close #134 then

Xeyame avatar Oct 01 '19 21:10 Xeyame

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!

mdcfe avatar Oct 02 '19 10:10 mdcfe

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.

Xeyame avatar Oct 02 '19 11:10 Xeyame

Needs unit tested. -1

Ichbinjoe avatar Oct 02 '19 18:10 Ichbinjoe