RFC: hello-world: remove reference to "name not given"#519
Closed
petertseng wants to merge 1 commit intoexercism:masterfrom
petertseng:hello-world
Closed
RFC: hello-world: remove reference to "name not given"#519petertseng wants to merge 1 commit intoexercism:masterfrom petertseng:hello-world
petertseng wants to merge 1 commit intoexercism:masterfrom
petertseng:hello-world
Conversation
Member
Author
|
There is too much going on in this PR. I'll extract some of it in an issue, and make it applicable to all tracks, because as it stands everyone can fairly think that I'm imagining this problem and the problem has nothing to do with them, so it is not worth the time to read this PR. |
This implements the second option "change the hello-world README to be so vague..." of #520. As stated in #520, the point of hello-world is to give a very easy exercise. This helps give an intro to how to fetch, how to submit, and how to run the test suite for a given language. The README included with hello-world helps out with that. The clause "If a name is not given" is very difficult for languages that do not support overloading functions and/or default function parameters. This is against the vision of a very easy hello-world. The ideal is a function that returns "Hello, World!" unconditionally. By not mentioning the "name not given" in the README, every track that currently has a hello-world exercise can change it or not, as they please. This is a very delicate balance to strike. If we make it too vague, that is also undesirable. See #322. I cannot be sure that I have gotten it right in this commit.
Contributor
|
I don't think this is an ideal solution to the main problem with hello world. See #520 |
Member
Author
|
Let's close to avoid accidental merges while we examine our options in #520. |
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
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.
This implements the second option "change the hello-world README to be
so vague..." of #520.
As stated in #520, the point of hello-world is to give a very easy
exercise. This helps give an intro to how to fetch, how to submit, and
how to run the test suite for a given language. The README included with
hello-world helps out with that.
The clause "If a name is not given" is very difficult for languages that
do not support overloading functions and/or default function parameters.
This is against the vision of a very easy hello-world. The ideal is a
function that returns "Hello, World!" unconditionally.
By not mentioning the "name not given" in the README, every track that
currently has a hello-world exercise can change it or not, as they
please.
This is a very delicate balance to strike. If we make it too vague, that
is also undesirable. See #322. I cannot be sure that I have gotten it
right in this commit.