On Tue, Feb 12, 2019 at 12:24:40PM +0100, ismael@xxxxxxxxxxx wrote: > These bindings delete the word before and after the cursor > respectively. It didn't seem necessary to describe the thing in detail > because it has been commonplace for more than 30 years AFAICT. I'm sure that these key bindings are familiar to you and probably to a lot of people, but I wasn't aware of them. I imagine there might be other people who aren't aware of them as well. The commit message is an opportunity for you to explain both what your patch does and argue why it's valuable. It's also rather uncommon for commit messages in this project to lack a body; we typically prefer to say at least a few sentences about why we've chosen to make a change, even preferring to perhaps state some of the more obvious things just for clarity's sake to assist other people (and future us) in understanding what's going on. This isn't me trying to be difficult, but rather to help you get your patch in a state that's most likely to be accepted. It's common here to do a couple of revisions of a patch, incorporating reviewers' comments and polishing along the way. -- brian m. carlson: Houston, Texas, US OpenPGP: https://keybase.io/bk2204
Attachment:
signature.asc
Description: PGP signature
