sum-of-multiples: update description#476
sum-of-multiples: update description#476petertseng merged 1 commit intoexercism:masterfrom robphoenix:update/sum-of-mulitples-readme
Conversation
description updated to mention limits, exercism/go#340 (comment)
petertseng
left a comment
petertseng
left a comment
There was a problem hiding this comment.
This seems good. I have one note about possible redundancy. But if nobody thinks it is useful to change, I would just merge this as-is in a day or so.
|
|
||
| Write a program that can find the sum of the multiples of a given set of | ||
| numbers. | ||
| Write a program that, given a number, can find the sum of the multiples |
There was a problem hiding this comment.
This same line is in https://github.com/exercism/x-common/blob/master/exercises/sum-of-multiples/metadata.yml#L2 - and since the blurb gets prepended to the README (I discuss at #469 (comment)), that means this line will sandwich the README.
That is still better than the current situation where the README has two contradictory lines. There is also talk of NOT prepending the blurb.
One alternative could be to remove this line completely, leaving the prepend to do the explaining. Or we could leave it. Someone may eventually request to remove the redundancy, as happened in #386
We accepted the duplication in December 2016 in #476 Today we decide it's better not to duplicate.
2 participants
description updated to mention limits,
exercism/go#340 (comment)