Collision of question and solution in 'Implement map with reduce'

[robinpokorny]

Current task for Implement map with reduce is to write “a simple version of Array#map”. That corresponds with provided boilerplate.

However, the official solution implements even the second argument. This could be confusing.

I propose to keep the simple version.

This matter was discussed and fixed in #35. It was, unfortunately, introduced again in #81.

[PWAckerman]

To put another way, the boilerplate in the problem is misleading, as the official solution differs considerably in both the name of the function(Boilerplate: name is "arrayMap", solution: name is just "map") and the number of arguments (2 in the boilerplate, 3 in the solution).

[robinpokorny]

Yes, you could put it that way. I still find the task and boilerplate the genuine part and propose to edit the solution.

[timoxley]

Ideally, we could split it into two tasks. One to implement the simple version, second to implement the thisArg handling.

[robinpokorny]

@timoxley That is a good idea. And I think it would be a nice learning experience as well.