Hi, Alangi Derick <alangiderick@xxxxxxxxx> writes: > This is a patch to fix the problem as given in the site: > https://git.wiki.kernel.org/index.php/SmallProjectsIdeas Please have a look at Documentation/SubmittingPatches, on how to submit well formed patches. In particular, the following things should be improved: - The commit should have a meaningful commit message (See section 2 in SubmittingPatches) - The patch should be created with git format-patch, so it can be applied using git am. (See section 3 and 4) - Your work should be signed off, otherwise it can not be applied to the project. (See section 5) > diff --git a/help.c b/help.c > index 2072a87..e347bbc 100644 > --- a/help.c > +++ b/help.c > @@ -229,9 +229,18 @@ void list_common_cmds_help(void) > > puts(_("The most commonly used git commands are:")); > for (i = 0; i < ARRAY_SIZE(common_cmds); i++) { > - printf(" %s ", common_cmds[i].name); > - mput_char(' ', longest - strlen(common_cmds[i].name)); > - puts(_(common_cmds[i].help)); > + if(i%3 == 1){ > + printf(" %s ", common_cmds[i].name); > + mput_char(' ', longest - strlen(common_cmds[i].name)); > + puts(_(common_cmds[i].help)); > + printf("\n"); > + } > + else { > + printf(" %s ", common_cmds[i].name); > + mput_char(' ', longest - strlen(common_cmds[i].name)); > + puts(_(common_cmds[i].help)); > + } > + Patch formatting issues aside, I don't think this does what is suggested in the project. You are just adding a blank line after three lines in the help output, which I don't think improves the current state by much. The bzr help output on the other hand groups similar commands together, and shows them to the user in groups. So the first step would be to find out which commands should be grouped together and then to output the groups here. > } > } > > > Regards > Alangi Derick Ndimnain > -- > 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 -- 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