-
Notifications
You must be signed in to change notification settings - Fork 252
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid please in error messages #8141
Comments
EDIT: TL;DR - Please should be avoided because its not necessary in the context of reporting an error or providing the user with a set of options to choose from. A program is not a person (yet, see "corporations"), and does not have to display any humanizing features like politeness (or verbal abuse). "Please" has nothing to do with "optional", so that style guide is picking the wrong reason for excluding the word. Kindness and/or courtesy is not a bug. The premise is. Most uses of the word "please" do not communicate something being entirely optional, there is always a consequence. Usually "please" is followed with a direct or implied command. (EDIT: and implied or explicit consequence)
...or the flight will not land on time, and you will probably go to jail. You always have a choice, even with a command given to you by the TSA agent pointing the taser at you. There are times when options have less consequence.
... Or else? "Or else" in this example has been shown to have negligible - but existing - consequence. |
Per the Microsoft style guide: |
Motivation from: https://github.com/dotnet/cli/issues/11336
https://docs.microsoft.com/en-us/windows/desktop/debug/error-message-guidelines
We have ~139 please mentions in our resx files (about ~80 are not in the comment field).
All of these need reviewed and potentially fixed.
Examples
https://github.com/NuGet/NuGet.Client/blob/dev/src/NuGet.Core/NuGet.Commands/Strings.resx#L393
https://github.com/NuGet/NuGet.Client/blob/dev/src/NuGet.Core/NuGet.Commands/Strings.resx#L435
https://github.com/NuGet/NuGet.Client/blob/dev/src/NuGet.Core/NuGet.Commands/Strings.resx#L606
The text was updated successfully, but these errors were encountered: