-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
#770 - Removed all mentions of "please" from docs and messages #5040
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Grace Choi <[email protected]> Signed-off-by: Pranjal Rai <[email protected]>
What's wrong with having "please"? |
@vvoland I think this PR is to address; There's various articles on this, but here's a nice one that outlines some reasoning; https://medium.com/@xupeilan05/why-please-has-no-place-in-ux-writing-tips-for-clear-and-assertive-copy-bd096b7f976a |
Thank you for contributing! It appears your commit message is missing a DCO sign-off, We require all commit messages to have a
There is no need to open a new pull request, but to fix this (and make CI pass), Unfortunately, it's not possible to do so through GitHub's web UI, so this needs You can find some instructions in the output of the DCO check (which can be found Steps to do so "roughly" come down to:
Sorry for the hassle (I wish GitHub would make this a bit easier to do), and let me know if you need help or more detailed instructions! |
OH! I think the DCO may be the DCO check being too restrictive;
It's probably fine to mark that as "ok" manually; let me do so. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5040 +/- ##
==========================================
- Coverage 61.08% 61.08% -0.01%
==========================================
Files 295 298 +3
Lines 20660 20672 +12
==========================================
+ Hits 12621 12628 +7
- Misses 7142 7147 +5
Partials 897 897 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like this PR is modifying the vendor/
directory (see my comment)
Please see the LICENSE file for licensing information. | ||
See the LICENSE file for licensing information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh! Files inside the vendor/
directory are dependencies we use, but those are not maintained by us, and none of the files inside the vendor/
directory should be edited by us.
Can you revert changes inside vendor/
?
- What I did
We removed all references to the word "please" throughout the entire repository.
- How I did it
We used the search feature to look for all mentions of "please" and removed them. We also capitalized the following word if the "please" was the first word in the sentence.
- How to verify it
Use the search feature to look for instances of "please" in the code base.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)