Allow the use of `List.foldr` in `top-scorers`

exercism / elm-analyzer

GNU Affero General Public License v3.0

1 stars 4 forks source link

Allow the use of `List.foldr` in `top-scorers` #62

Closed jiegillet closed 12 months ago

jiegillet commented 1 year ago

Closes #56

I didn't change the analyzer comment

aggregateScorers is most easily written using List.foldl and updateGoalCountForPlayer.

because it's still true, but should I? If someone uses List.foldr but not updateGoalCountForPlayer, would they be confused and change both? It's not a huge deal I guess.

jiegillet commented 12 months ago

Opened a PR for the copy update: https://github.com/exercism/website-copy/pull/2288

I don't think it's fundamentally blocking this PR though, so I'll merge it now.