accumulate: remove Lisp-specific portion #871
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Elixir-specific portion added: exercism/exercism#1136 Lisp-specific portion added: #98 Elixir-specific portion removed because the `Enum.reduce` was not necessary: #274 The reason to suggest removing this Lisp-specific now is slightly different: It doesn't seem to make much sense to include this Lisp-specific sentence in non-Lisp tracks. See that http://x.exercism.io/problems/accumulate indicates this exercise is only implemented by one Lisp-like language (Clojure) and 19 non-Lisp-like languages. It could be possible for the tracks that desire this clause to place it in .meta/HINTS.md. It is easy to **add** content to the description.md, but it is not easy to **remove** content. Leaving the content in description.md is potentially asking 19 different tracks to take custom measures to remove it (an exercise-specific template), unless we come up with a better way to **remove** content (a .patch to be applied to each description.md, perhaps). It is acknowledged that #129 indicates the exercise will be deprecated at a point in the future. Nevertheless, it is still true that the exercise exists, and that new implementations for it are being contributed (exercism/rust#332) and that this README change benefits those tracks that have this exercise.
ErikSchierboom
approved these changes
Aug 4, 2017
This was referenced Aug 22, 2017
ijanos
pushed a commit
to exercism/rust
that referenced
this pull request
Sep 1, 2017
Elixir-specific portion added: exercism/exercism#1136 Lisp-specific portion added: exercism/problem-specifications#98 Elixir-specific portion removed because the `Enum.reduce` was not necessary: exercism/problem-specifications#274 It doesn't seem to make much sense to include this Lisp-specific sentence in non-Lisp tracks. exercism/problem-specifications#871
petertseng
added a commit
to exercism/haskell
that referenced
this pull request
Sep 8, 2017
Elixir-specific portion added: exercism/exercism#1136 Lisp-specific portion added: exercism/problem-specifications#98 Elixir-specific portion removed because the `Enum.reduce` was not necessary: exercism/problem-specifications#274 It doesn't seem to make much sense to include this Lisp-specific sentence in non-Lisp tracks. exercism/problem-specifications#871
emcoding
pushed a commit
that referenced
this pull request
Nov 19, 2018
change: Update generator & Regenerate Tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Elixir-specific portion added:
exercism/exercism#1136
Lisp-specific portion added:
#98
Elixir-specific portion removed because the
Enum.reducewas notnecessary:
#274
The reason to suggest removing this Lisp-specific now is slightly
different:
It doesn't seem to make much sense to include this Lisp-specific
sentence in non-Lisp tracks.
See that http://x.exercism.io/problems/accumulate indicates this
exercise is only implemented by one Lisp-like language (Clojure) and 19
non-Lisp-like languages.
It could be possible for the tracks that desire this clause to place it
in .meta/HINTS.md. It is easy to add content to the description.md,
but it is not easy to remove content. Leaving the content in
description.md is potentially asking 19 different tracks to take custom
measures to remove it (an exercise-specific template), unless we come up
with a better way to remove content (a .patch to be applied to each
description.md, perhaps).
It is acknowledged that
#129 indicates
the exercise will be deprecated at a point in the future. Nevertheless,
it is still true that the exercise exists, and that new implementations
for it are being contributed (exercism/rust#332)
and that this README change benefits those tracks that have this
exercise.