Skip to content

Update instructions.append.md#741

Merged
glennj merged 2 commits intoexercism:mainfrom
OmkarBhosekar7711:patch-1
Jun 25, 2025
Merged

Update instructions.append.md#741
glennj merged 2 commits intoexercism:mainfrom
OmkarBhosekar7711:patch-1

Conversation

@OmkarBhosekar7711
Copy link
Contributor

  • Fix typo and clarify instructions in usage text
  • Corrected a typo ("treat is" ➝ "treat it") in the program instructions.
  • Added the word "code" at the end of the sentence "exit with a non-zero status" for clarity.

pull request template

If the program is run with exactly one argument (even if it is an empty string), treat it as a person's name and print a greeting message.
If it is run with zero arguments or more than one argument, print an error message and exit with a non-zero status code.


Reviewer Resources:

Track Policies

- Fix typo and clarify instructions in usage text
- Corrected a typo ("treat is" ➝ "treat it") in the program instructions.
- Added the word "code" at the end of the sentence "exit with a non-zero status" for clarity.
@glennj
Copy link
Contributor

glennj commented Jun 24, 2025

I appreciate you finding the typo.

"your" -> "the" is debatable.

I disagree with "exit status code" -- "exit status" is the generally accepted term, I think.

- Fix typo and clarify instructions in usage text
- Corrected a typo ("treat is" ➝ "treat it") in the program instructions.
@OmkarBhosekar7711
Copy link
Contributor Author

I have updated the pull request with the changes you suggested. Kindly review my latest commit and let me know if there is any feedback or further improvements I should make.

@glennj glennj added x:rep/small Small amount of reputation x:type/docs Work on Documentation labels Jun 25, 2025
@glennj glennj merged commit 9611d28 into exercism:main Jun 25, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:rep/small Small amount of reputation x:type/docs Work on Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants