Explv / Explvs-AIO

Explv's AIO
MIT License
23 stars 18 forks source link

Cooks assistant #21

Closed ghost closed 5 years ago

ghost commented 5 years ago

Added the quest cooks assistant, this will complete cooks assistants without any requirements.

ghost commented 5 years ago

Do you want me to fix these two issues that codacy came up with?

  1. There was no default statement in the switch in the example, I could make the default throw an exception or something.
  2. I have never heard about "NPath complexity" but I can split things out into their own functions and such, I just find the commenting fine.
Explv commented 5 years ago

@RealPoke I will review this PR this evening, thanks for the contribution

Explv commented 5 years ago

Logic was all there, just needed a bit of refactoring.

Thanks again for the contribution!