Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(6369)

Issue 566930043: doc: Update section on "commit access" (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 6 months ago by hahnjo
Modified:
4 years, 3 months ago
Reviewers:
Carl, carl.d.sorensen, lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

doc: Update section on "commit access" * Savannah doesn't use CAcert anymore. If the user faces an untrusted connection, importing the CAcert root certificate does not help. * Recommend "git remote set-url" instead of a direct "git config".

Patch Set 1 #

Total comments: 1

Patch Set 2 : Remove unneeded @tie in @command #

Total comments: 1

Patch Set 3 : Remove unneeded @w{} #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -19 lines) Patch
M Documentation/contributor/source-code.itexi View 1 2 7 chunks +8 lines, -19 lines 0 comments Download

Messages

Total messages: 6
lemzwerg
LGTM https://codereview.appspot.com/566930043/diff/581200043/Documentation/contributor/source-code.itexi File Documentation/contributor/source-code.itexi (right): https://codereview.appspot.com/566930043/diff/581200043/Documentation/contributor/source-code.itexi#newcode2196 Documentation/contributor/source-code.itexi:2196: The @command{git@tie{}remote@tie{}set-url} command above should modify your I ...
4 years, 6 months ago (2019-10-24 19:05:49 UTC) #1
hahnjo
Remove unneeded @tie in @command
4 years, 6 months ago (2019-10-25 13:02:02 UTC) #2
hahnjo
On 2019/10/24 19:05:49, lemzwerg wrote: > LGTM > > https://codereview.appspot.com/566930043/diff/581200043/Documentation/contributor/source-code.itexi > File Documentation/contributor/source-code.itexi (right): > ...
4 years, 6 months ago (2019-10-25 13:03:19 UTC) #3
lemzwerg
Thanks for the corrections. https://codereview.appspot.com/566930043/diff/551130043/Documentation/contributor/source-code.itexi File Documentation/contributor/source-code.itexi (right): https://codereview.appspot.com/566930043/diff/551130043/Documentation/contributor/source-code.itexi#newcode2217 Documentation/contributor/source-code.itexi:2217: @command{git config @w{push.default} matching} command ...
4 years, 6 months ago (2019-10-25 17:02:23 UTC) #4
Carl
LGTM
4 years, 6 months ago (2019-10-25 19:21:53 UTC) #5
hahnjo
4 years, 6 months ago (2019-10-26 09:07:08 UTC) #6
Remove unneeded @w{}
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b