Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes: > Elia Pinto <gitter.spiros@xxxxxxxxx> writes: > >> The Git CodingGuidelines prefer the $(...) construct for command >> substitution instead of using the backquotes `...`. > > For patches 1 to 14: > > Reviewed-by: Matthieu Moy <Matthieu.Moy@xxxxxxx> > > (reviewed the patches in my mailer, and the "diff --color-words=." after > applying in addition) Thanks both. Ideally, we should keep these scripted Porcelain implementations in the contrib/examples/ hierarchy up-to-date to still work with the recent versions of Git as they used to. We do not need to backport new features, but they should follow the best-current-practice of the use of plumbing features and scripting in general, and this update is probably a step in the right direction. It is somewhat sad that these are not tested but I do not think of an unintrusive easy way to keep them also in the test suite. Will queue. -- 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