Matthieu Moy <matthieu....@grenoble-inp.fr> writes: > Michal Nazarewicz <m...@google.com> writes: > >> From: Michal Nazarewicz <min...@mina86.com> >> >> Add a credential() function which is an interface to the git >> credential command. The code is heavily based on credential_* >> functions in <contrib/mw-to-git/git-remote-mediawiki>. > > I'm no perl expert, so I cannot comment much on style (there are many > small changes compared to the mediawiki code that look like improvement > though), but: > > Reviewed-by: Matthieu Moy <matthieu....@imag.fr>
Thanks. I'd actually be more worried about the error checking issue Peff raised during his review. I have a feeling that "when in doubt, do not cause harm" is a more prudent way to go than "I do not know, so I'll let anything pass". -- 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
