Documentation/gerrit_guidelines: Fix newlines
Markdown does not render newlines unless there is an empty line in between the lines of text. Several command examples and a list were missing these empty lines, causing their content to be rendered inline with the preceding text. Fix this by adding triple backticks around code blocks and bullet points to rows of text in a list. Change-Id: I9c1d2b81acdeb378346c68bced0cdbfeeb81bf26 Signed-off-by: Nicholas Chin <nic.c3.14@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/72625 Reviewed-by: Elyes Haouas <ehaouas@noos.fr> Reviewed-by: Felix Singer <felixsinger@posteo.net> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
parent
7cba1c486b
commit
3bce5643ca
|
@ -173,7 +173,10 @@ This helps verify that the patch train won’t tie up the jenkins builders
|
||||||
for no reason if there are failing patches in the train. For running
|
for no reason if there are failing patches in the train. For running
|
||||||
parallel builds, you can specify the number of cores to use by setting the
|
parallel builds, you can specify the number of cores to use by setting the
|
||||||
the CPUS environment variable. Example:
|
the CPUS environment variable. Example:
|
||||||
make what-jenkins-does CPUS=8
|
|
||||||
|
```Bash
|
||||||
|
make what-jenkins-does CPUS=8
|
||||||
|
```
|
||||||
|
|
||||||
* Use a topic when pushing a train of patches. This groups the commits
|
* Use a topic when pushing a train of patches. This groups the commits
|
||||||
together so people can easily see the connection at the top level of
|
together so people can easily see the connection at the top level of
|
||||||
|
@ -181,7 +184,10 @@ gerrit. Topics can be set for individual patches in gerrit by going into
|
||||||
the patch and clicking on the icon next to the topic line. Topics can also
|
the patch and clicking on the icon next to the topic line. Topics can also
|
||||||
be set when you push the patches into gerrit. For example, to push a set of
|
be set when you push the patches into gerrit. For example, to push a set of
|
||||||
commits with the i915-kernel-x60 set, use the command:
|
commits with the i915-kernel-x60 set, use the command:
|
||||||
git push origin HEAD:refs/for/master%topic=i915-kernel-x60
|
|
||||||
|
```Bash
|
||||||
|
git push origin HEAD:refs/for/master%topic=i915-kernel-x60
|
||||||
|
```
|
||||||
|
|
||||||
* If one of your patches isn't ready to be merged, make sure it's obvious
|
* If one of your patches isn't ready to be merged, make sure it's obvious
|
||||||
that you don't feel it's ready for merge yet. The preferred way to show
|
that you don't feel it's ready for merge yet. The preferred way to show
|
||||||
|
@ -191,7 +197,10 @@ Examples of this are "WIP: title" or "[NEEDS_TEST]: title". Another way to
|
||||||
mark the patch as not ready would be to give it a -1 or -2 review, but
|
mark the patch as not ready would be to give it a -1 or -2 review, but
|
||||||
isn't as obvious as the commit message. These patches can also be pushed with
|
isn't as obvious as the commit message. These patches can also be pushed with
|
||||||
the wip flag:
|
the wip flag:
|
||||||
git push origin HEAD:refs/for/master%wip
|
|
||||||
|
```Bash
|
||||||
|
git push origin HEAD:refs/for/master%wip
|
||||||
|
```
|
||||||
|
|
||||||
* When pushing patches that are not for submission, these should be marked
|
* When pushing patches that are not for submission, these should be marked
|
||||||
as such. This can be done in the title ‘[DONOTSUBMIT]’, or can be pushed as
|
as such. This can be done in the title ‘[DONOTSUBMIT]’, or can be pushed as
|
||||||
|
@ -200,10 +209,16 @@ sorts of patches are frequently posted as ideas or RFCs for the community to
|
||||||
look at. Note that private changes can still be fetched from Gerrit by anybody
|
look at. Note that private changes can still be fetched from Gerrit by anybody
|
||||||
who knows their commit ID, so don't use this for sensitive changes. To push
|
who knows their commit ID, so don't use this for sensitive changes. To push
|
||||||
a private change, use the command:
|
a private change, use the command:
|
||||||
git push origin HEAD:refs/for/master%private
|
|
||||||
|
```Bash
|
||||||
|
git push origin HEAD:refs/for/master%private
|
||||||
|
```
|
||||||
|
|
||||||
* Multiple push options can be combined:
|
* Multiple push options can be combined:
|
||||||
git push origin HEAD:refs/for/master%private,wip,topic=experiment
|
|
||||||
|
```Bash
|
||||||
|
git push origin HEAD:refs/for/master%private,wip,topic=experiment
|
||||||
|
```
|
||||||
|
|
||||||
* Respond to anyone who has taken the time to review your patches, even if
|
* Respond to anyone who has taken the time to review your patches, even if
|
||||||
it's just to say that you disagree. While it may seem annoying to address a
|
it's just to say that you disagree. While it may seem annoying to address a
|
||||||
|
@ -277,13 +292,15 @@ git/gerrit tags by prepending the lines with 'Original-'. Marking
|
||||||
the original text this way makes it much easier to tell what changes
|
the original text this way makes it much easier to tell what changes
|
||||||
happened in which repository. This applies to these lines, not the actual
|
happened in which repository. This applies to these lines, not the actual
|
||||||
commit message itself:
|
commit message itself:
|
||||||
Commit-Id:
|
|
||||||
Change-Id:
|
* Commit-Id:
|
||||||
Signed-off-by:
|
* Change-Id:
|
||||||
Reviewed-on:
|
* Signed-off-by:
|
||||||
Tested-by:
|
* Reviewed-on:
|
||||||
Reviewed-by:
|
* Tested-by:
|
||||||
The script 'util/gitconfig/rebase.sh' can be used to help automate this.
|
* Reviewed-by:
|
||||||
|
|
||||||
|
The script `util/gitconfig/rebase.sh` can be used to help automate this.
|
||||||
Other tags such as 'Commit-Queue' can simply be removed.
|
Other tags such as 'Commit-Queue' can simply be removed.
|
||||||
|
|
||||||
* Check if there's documentation that needs to be updated to remain current
|
* Check if there's documentation that needs to be updated to remain current
|
||||||
|
|
Loading…
Reference in New Issue