[coreboot-gerrit] [M] Change in coreboot[master]: Documentation/gerrit_guidelines: Fix newlines

[Elyes Haouas (Code Review)]

Elyes Haouas has submitted this change. ( https://review.coreboot.org/c/coreboot/+/72625 )

Change subject: Documentation/gerrit_guidelines: Fix newlines
......................................................................

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 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72625
Reviewed-by: Elyes Haouas 
Reviewed-by: Felix Singer 
Tested-by: build bot (Jenkins) 
---
M Documentation/contributing/gerrit_guidelines.md
1 file changed, 51 insertions(+), 12 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Elyes Haouas: Looks good to me, approved
  Felix Singer: Looks good to me, approved

diff --git a/Documentation/contributing/gerrit_guidelines.md b/Documentation/contributing/gerrit_guidelines.md
index 74f79c0..e23bff7 100644
--- a/Documentation/contributing/gerrit_guidelines.md
+++ b/Documentation/contributing/gerrit_guidelines.md
@@ -173,7 +173,10 @@
 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
 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
 together so people can easily see the connection at the top level of
@@ -181,7 +184,10 @@
 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
 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
 that you don't feel it's ready for merge yet. The preferred way to show
@@ -191,7 +197,10 @@
 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
 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
 as such. This can be done in the title ‘[DONOTSUBMIT]’, or can be pushed as
@@ -200,10 +209,16 @@
 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
 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:
-        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
 it's just to say that you disagree. While it may seem annoying to address a
@@ -277,13 +292,15 @@
 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
 commit message itself:
-        Commit-Id:
-        Change-Id:
-        Signed-off-by:
-        Reviewed-on:
-        Tested-by:
-        Reviewed-by:
-The script 'util/gitconfig/rebase.sh' can be used to help automate this.
+
+	* Commit-Id:
+	* Change-Id:
+	* Signed-off-by:
+	* Reviewed-on:
+	* Tested-by:
+	* 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.

* Check if there's documentation that needs to be updated to remain current