Pranit Bauva <pranit.ba...@gmail.com> writes: > A very interesting fact to note: > I made a mistake by dropping of the first argument 'term' in the function > one_of() and compiled and the test suite passed fully (which was pointed > out by Christian). This shows that the test coverage is incomplete. > I am > currently writing tests to improve on that coverage. It will be included > separately. > > Link to v4: http://thread.gmane.org/gmane.comp.version-control.git/293488 > > Changes wrt v4: > * Stick with 'subcommand' in the commit message. > * Rename enum as 'subcommand' to make it singular.
I've already said what needs to be said on this. > * s/bug/BUG/g > * Drop _() in the default of switch case > * Use some suggestions for commit message no. 2 and also include why I am > using subcommand. I am not particularly impressed by the rationale, though. As a starter project to show that you learned how to add a new mode to bisect--helper, check-term-format may be a small enough function that is easy to dip the toe into the codebase, so in that sense it may be an appropriate first step, but otherwise it is not all that interesting, especially when we _know_ that it will be discarded. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
