search

exercism / coffeescript

Exercism exercises in CoffeeScript.
https://exercism.org/tracks/coffeescript
MIT License
19 stars 50 forks source link

Update grains test suite #249

Closed BNAndras closed 6 months ago

BNAndras commented 6 months ago

@ErikSchierboom, @ladokp noticed an issue where I updated the stub from your code review but not the CI example. If I had, the CI wouldn't have passed. I also removed the unused argument on the last test, a copy and paste error.

ladokp commented 6 months ago

This should fix the error in my opinion. Also the other new exercises are facing the same issue. I have commented in the corresponding pull requests.

ladokp commented 6 months ago

I've created PR #250 to fix the test suite. Will also fix the other exercises.

ladokp commented 6 months ago

PR #251, #252, #253 were added.