<p>Michael Bacarella has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30060">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Documentation/lessons/lesson2.md: clear up confusing language around running gitconfig<br><br>It's easy to misinterpret or miss altogether the instruction to run 'make<br>gitconfig', which will cause strange problems a few commands later.  Revise the<br>docuentation to make it clearer.  Also adds a blurb further down with a link to<br>find Gerrit workflow docs.<br><br>Change-Id: I49734c724c4d6da716a358cd849938ef14dab3b1<br>Signed-off-by: Michael Bacarella <michael.bacarella@gmail.com><br>---<br>M Documentation/lessons/lesson2.md<br>1 file changed, 14 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/30060/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/Documentation/lessons/lesson2.md b/Documentation/lessons/lesson2.md</span><br><span>index a95dd80..5b774cd 100644</span><br><span>--- a/Documentation/lessons/lesson2.md</span><br><span>+++ b/Documentation/lessons/lesson2.md</span><br><span>@@ -71,14 +71,18 @@</span><br><span> If you are using HTTP, instead, select **http** from the tabs under "Project coreboot"</span><br><span> and run the command that appears</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-After it finishes cloning, "cd coreboot" will take you into the local</span><br><span style="color: hsl(0, 100%, 40%);">-git repository. Run "make gitconfig" to set up the hooks and configurations.</span><br><span style="color: hsl(0, 100%, 40%);">-For example, you will be asked to run the following commands to set your</span><br><span style="color: hsl(0, 100%, 40%);">-username and email.</span><br><span style="color: hsl(120, 100%, 40%);">+Now is a good time to configure your global git identity, if you haven't</span><br><span style="color: hsl(120, 100%, 40%);">+already.</span><br><span> </span><br><span>     git config --global user.name "Your Name"</span><br><span>  git config --global user.email "Your Email"</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+Finally, enter the local git repository and set up repository specific hooks</span><br><span style="color: hsl(120, 100%, 40%);">+and other configurations.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+  cd coreboot</span><br><span style="color: hsl(120, 100%, 40%);">+   make gitconfig</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> ## Part 4: Submit a commit</span><br><span> </span><br><span> An easy first commit to make is fixing existing checkpatch errors and warnings</span><br><span>@@ -208,6 +212,12 @@</span><br><span> Submitting as a draft means that your commit will be on coreboot.org, but is</span><br><span> only visible to those you add as reviewers.</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+This has been a quick primer on how to submit a change to Gerrit for review</span><br><span style="color: hsl(120, 100%, 40%);">+using git.  You may wish to review the Gerrit code review workflow</span><br><span style="color: hsl(120, 100%, 40%);">+documentation, especially if you plan to work on multiple changes at the same</span><br><span style="color: hsl(120, 100%, 40%);">+time.  See</span><br><span style="color: hsl(120, 100%, 40%);">+https://gerrit-review.googlesource.com/Documentation/intro-user.html#code-review</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> ## Part 5: Getting your commit reviewed</span><br><span> </span><br><span> Your commits can now be seen on review.coreboot.org if you select “Your”</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30060">change 30060</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30060"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I49734c724c4d6da716a358cd849938ef14dab3b1 </div>
<div style="display:none"> Gerrit-Change-Number: 30060 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Michael Bacarella <michael.bacarella@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>