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

Issue 291160043: Doc: CG - Re-organize information about 'Patches' (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years ago by pkx166h
Modified:
8 years ago
Reviewers:
pwm
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Doc: CG - Re-organize information about 'Patches' Stage 1 of 2 The information about how to create, manage, upload and review patches (as well as the 'Patch cycle') is in many different places making it awkward (and time consuming) for devs - especially those not used to the process - to locate the information. Some of the information for 'using git' to create patches does overlap with informeation on how to submit patches. This results in a lot of unnecessary 'jumping' about the documentation. This patch simply moves sections - no new information has been added - although some of the @node and @*section names have been changed to help with re-organization.

Patch Set 1 #

Total comments: 3

Patch Set 2 : With Paul's suggestions. Thanks. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+253 lines, -255 lines) Patch
M Documentation/contributor/administration.itexi View 1 1 chunk +2 lines, -1 line 0 comments Download
M Documentation/contributor/issues.itexi View 1 2 chunks +4 lines, -213 lines 0 comments Download
M Documentation/contributor/source-code.itexi View 1 9 chunks +247 lines, -41 lines 0 comments Download

Messages

Total messages: 6
pwm
Good idea to consolidate the patch information. My suggestion is to just to keep sections ...
8 years ago (2016-03-13 17:04:04 UTC) #1
pwm
Another possibility. -Paul https://codereview.appspot.com/291160043/diff/1/Documentation/contributor/source-code.itexi File Documentation/contributor/source-code.itexi (right): https://codereview.appspot.com/291160043/diff/1/Documentation/contributor/source-code.itexi#newcode853 Documentation/contributor/source-code.itexi:853: * The patch review cycle:: Another ...
8 years ago (2016-03-14 14:55:01 UTC) #2
pkx166h
With Paul's suggestions. Thanks.
8 years ago (2016-03-17 18:15:05 UTC) #3
pkx166h
Thanks Paul. https://codereview.appspot.com/291160043/diff/1/Documentation/contributor/source-code.itexi File Documentation/contributor/source-code.itexi (right): https://codereview.appspot.com/291160043/diff/1/Documentation/contributor/source-code.itexi#newcode853 Documentation/contributor/source-code.itexi:853: * The patch review cycle:: On 2016/03/13 ...
8 years ago (2016-03-17 18:15:29 UTC) #4
pwm
LGTM. On 2016/03/17 18:15:29, pkx166h wrote: > > I've opted for this > > * ...
8 years ago (2016-03-17 18:37:49 UTC) #5
pkx166h
8 years ago (2016-03-22 16:58:39 UTC) #6
author	James Lowe <pkx166h@gmail.com>	
	Tue, 8 Mar 2016 12:09:50 +0000 (12:09 +0000)
committer	James Lowe <pkx166h@gmail.com>	
	Tue, 22 Mar 2016 16:55:11 +0000 (16:55 +0000)
commit	b3e454d64b973f0bb2a86e49dd49d5a01df59164

Stage 2 will be in a different Reitveld but with the same Tracker number
Sign in to reply to this message.

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