exercism / elm-analyzer

GNU Affero General Public License v3.0
1 stars 4 forks source link

Write analysis for practice exercise `accumulate` #24

Closed jiegillet closed 11 months ago

jiegillet commented 2 years ago

The exercise description says:

Restrictions Keep your hands off that collect/map/fmap/whatchamacallit functionality provided by your standard library! Solve this one yourself using other basic tools instead.

The exercise can be solved with accumulate = List.map but should avoid the use of any List functions and promote implementing it with pattern matching and recursion instead. The example solution should also be changed accordingly.

jiegillet commented 11 months ago

This exercise has been deprecated, not worth adding an analysis for. The advice is to implement list-ops instead, which we have. @ceddlyburge do you think we should remove the exercise or not worry about it and leave it? AFAIK it's the track's choice.

mpizenberg commented 11 months ago

For what its worth, I personally do not like the accumulate exercise, and very much prefer the list-ops one. (just my 2 cents)

ceddlyburge commented 11 months ago

I agree, the list-ops one is better, and as you mention, this one is deprecated, so probably removing it is best. There are about 100 published solutions of it, so those people will lose those solutions, but I don't think that is a huge deal. Those people will almost certainly have done other solutions as well, so probably won't really notice.

jiegillet commented 11 months ago

Actually I misspoke, we shouldn't remove it, we should deprecate it, which is a non-destructive action (existing solutions remain, but the exercise is not visible to the ones who haven't solved it). Cool, let's deprecate it then.