Re: [Outreachy][PATCH 0/1] builtin/branch.c: ammend die() error message

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

 



On 2023-10-11 17:42, Isoken Ibizugbe wrote:
Thanks for the feedback. How do I add the description of the patch.

Could you, please, use inline replying? We've asked that at least four times already.

Regarding the description of the patch, just write it as the commit comment in your local git repository. Using git-format-patch will pick the commit comment up and put it into the email message file you'll then send to the mailing list using git-send-email. It's all rather simple and straightforward.


On Wed, 11 Oct 2023 at 4:34 PM, Dragan Simic <dsimic@xxxxxxxxxxx>
wrote:

On 2023-10-11 17:24, Isoken June Ibizugbe wrote:
This patch improves the consistency and clarity of error messages
across Git commands. It adheres to Git's Coding Guidelines for
error
messages:

- Error messages no longer end with a full stop.
- Capitalization is avoided in error messages.
- Error messages lead with a description of the issue, enhancing
readability.

Signed-off-by: Isoken June Ibizugbe <isokenjune@xxxxxxxxx>

When there's only one patch and not a series, there's no need for a
cover letter, and the description of the patch actually needs to go
into
the patch itself, so it can be latter pulled into the repository as
part
of the patch submission.

Isoken June Ibizugbe (1):
branch.c: ammend error messages for die()

builtin/branch.c | 38 +++++++++++++++++++-------------------
1 file changed, 19 insertions(+), 19 deletions(-)



[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]

  Powered by Linux