Re: [PATCH] gitweb(1) Fixed English issues

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



D Waitzman <djw@xxxxxxxxxxx> writes:

> This is a trivial patch to correct the english in the man page.  Ben Walton suggested an additional english fix which I have added and squashed with my change.  And then I added it again when he pointed out I missed something in his original email.
>
> --
>
> Fixed improper use of "it's" when it should be "its".
> Added missing "the"; noted by Ben Walton bwalton@xxxxxxxxxxxxxxxxxx
>
>     Signed-off-by: David Waitzman <djw@xxxxxxx>
> ---
>  Documentation/gitweb.txt |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/gitweb.txt b/Documentation/gitweb.txt
> index 605a085..168e8bf 100644
[snip patch]

Note that the correct way to put commentary in the patch mail that does
not go into the commit message is putting it after the --- marker.
Also, we write commit messages in the imperative.  The above should
probably look roughly like

  Subject: Documentation/gitweb: trivial English fixes

  Change "it's" to "its" where a possessive is intended.

  Also add two missing "the" that were noticed by Ben Walton
  <bwalton@xxxxxxxxxxxxxxxxxx>.

  Signed-off-by: David Waitzman <djw@xxxxxxx>
  ---
  This is a trivial patch to correct the english in the man page.  Ben
  Walton suggested an additional english fix which I have added and
  squashed with my change.  And then I added it again when he pointed
  out I missed something in his original email.

   Documentation/gitweb.txt |    4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)

  diff --git a/Documentation/gitweb.txt b/Documentation/gitweb.txt
  index 605a085..168e8bf 100644

Note that I added "Subject:" here to make it an inline patch (though you
obviously need to remove my extra formatting indent).  If there's no
Subject pseudoheader, the patch gets the subject of your email, and
similarly for From and author.

-- 
Thomas Rast
trast@{inf,student}.ethz.ch
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]