Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11896 )
Change subject: docs: Use a non-SSH method of installing Gerrit commit hook ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/11896/1/docs/contributing.adoc File docs/contributing.adoc: http://gerrit.cloudera.org:8080/#/c/11896/1/docs/contributing.adoc@72 PS1, Line 72: -L I think it's a bit more robust and less chatty with '-s -S' options (that might be preferable for scripts): curl -L -s -S https://.../commit-msg -o ... http://gerrit.cloudera.org:8080/#/c/11896/1/docs/contributing.adoc@72 PS1, Line 72: <username>@ I think it's possible to drop username, no? -- To view, visit http://gerrit.cloudera.org:8080/11896 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ibdb024aafcd601f740ae73d83c54f9659cc5a3a9 Gerrit-Change-Number: 11896 Gerrit-PatchSet: 1 Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 07 Nov 2018 04:07:05 +0000 Gerrit-HasComments: Yes