-
-
Notifications
You must be signed in to change notification settings - Fork 555
Simplifying HighScores #1456
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Not core (yet) for us (js/ts), but makes sense. I really like the simplicity of space-age and I think it'll helpt this as well so you can focus on the data structure instead of the formatting. |
I am fine either way. I may leave the |
I'm also in favor of removing the report cases, for the same reasons as @SleeplessByte: it allows focusing on the data structure. It will also make reviewing/auto-approving easier. |
This sounds great to me 👍 I'm ok with either removing the method entirely or finding a simpler replacement. Also found it to be a pain point for mentoring. |
Thanks all! @pgaspar we're going to tackle this, yes? |
Heads up: there's a PR up to remove the method, meaning the exercise focusses on array query methods. #1459 |
In the Ruby track, High Scores works great as an easy exercise for students, but the
report
method makes it a pain to mentor. We want to simplify it such that it doesn't need the giant if/else statement/extracted method/unreadable interpolation anymore.Is this interesting for more tracks? In that case, we can change the canonical data. If it's only an issue in Ruby, we'll change it only there.
It's in
@exercism/dart, @exercism/csharp @exercism/python @exercism/scala @exercism/delphi @exercism/javascript @exercism/fsharp ?(cc @pgaspar)
The text was updated successfully, but these errors were encountered: