On Tue, Feb 22, 2022 at 6:01 PM Shubham Mishra <shivam828787@xxxxxxxxx> wrote: > > pipes doesn't care about error codes and ignore them thus we should not use them in tests. > As an easy alternative, I am using a tmp file to write from git command so we can test the exit code. (Others have already rightly commented on many things, so I am only focusing on microproject related things.) As the subject of this cover letter starts with "microproject:" I would suggest taking a look (or maybe reading again) our page about microprojects: https://git.github.io/General-Microproject-Information/ Please note that we ask that all related emails start with “[GSoC]” or “[Outreachy]” or something similar, not "microproject:". > This is my first contribution that's why I am keeping diff short with an intention to understand the process instead of making impactful change in first attempt. Later, I will send more patches to fix the issue for other files. Please read the "Only ONE quality focused microproject per student" section of the above mentioned page. Thanks for your first contribution!
