go icon indicating copy to clipboard operation
go copied to clipboard

First Class Functions: task 5 misleading test?

Open randych-gilad opened this issue 1 year ago • 4 comments

At first it was like: https://github.com/exercism/go/blob/0419fde7a7d147bbe40f7b437a5d107c780eabb6/exercises/concept/expenses/.docs/instructions.md?plain=1#L144 But then: https://github.com/exercism/go/blob/0419fde7a7d147bbe40f7b437a5d107c780eabb6/exercises/concept/expenses/expenses_test.go#L224 which contradicts the "regardless" part. I think that's human error or just plain misleading.

randych-gilad avatar Mar 20 '24 17:03 randych-gilad

Hello. Thanks for opening an issue on Exercism 🙂

At Exercism we use our Community Forum, not GitHub issues, as the primary place for discussion. That allows maintainers and contributors from across Exercism's ecosystem to discuss your problems/ideas/suggestions without them having to subscribe to hundreds of repositories.

This issue will be automatically closed. Please use this link to copy your GitHub Issue into a new topic on the forum, where we look forward to chatting with you!

If you're interested in learning more about this auto-responder, please read this blog post.

github-actions[bot] avatar Mar 20 '24 17:03 github-actions[bot]

Would changing this to "returns error when no records with category found in any period" (removed the word "days") make it clearer for you?

andrerfcsantos avatar Jul 27 '24 18:07 andrerfcsantos

Yes, I believe so

randych-gilad avatar Aug 02 '24 19:08 randych-gilad

@randych-gilad I'll take a PR for this if you want to make one :)

andrerfcsantos avatar Aug 03 '24 17:08 andrerfcsantos