From: Shourya Shukla Date: Thu, 19 Mar 2020 16:38:17 GMT Subject: Re: [GSoC][PATCH 1/2] t4131: modernize style Message-ID: <20200319163817.4239-1-shouryashukla.oo@gmail.com> In-Reply-To: <20200319132957.17813-2-harshitjain1371999@gmail.com> Hello Harshit, > The tests in 't4131-apply-fake-ancestor.sh' were written a long time ago, and have a few style violations. Update it to adhere to the CodingGuidelines. Maybe add a commit title and then have a body? To do so, do a 'git commit' instead of 'git commit -m "message"'. This will open a text editor in which you can edit your commit message. You may refer to this answer I gave on StackOverflow on commit messages: https://stackoverflow.com/a/60755299/10751129 Also, commit messages are generally around 72 characters per line. What are the style violations you are talking about BTW? The commit title can be of the form: t4131: modernise style <> Regards, Shourya Shukla