<p>Looks great at a glance, <a class="user-mention" data-hovercard-user-id="735155" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/m52go">@m52go</a>, I'll provide more detailed comments later, but in the meantime, I've pushed a couple review commits, please see and study them above.</p>
<p>A few other nits:</p>
<ol>
<li>Pease read and follow our guidelines around commit comments at <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="319178416" data-permission-text="Issue title is private" data-url="https://github.com/bisq-network/style/issues/9" href="https://github.com/bisq-network/style/issues/9">bisq-network/style#9</a>.</li>
<li>Configure your Git client for proper author metadata (first and last name)</li>
<li>Set yourself up for GPG commit signing</li>
<li>In the future, please submit PRs from a branch other than your <code>master</code> branch, e.g. in this case, <code>getting-started</code> would have been fine. I'll omit the reasons for this here, but can explain more fully later if you like.</li>
</ol>
<p>Links to and further information about each of these things can be found in the <a href="https://docs.bisq.network/contributor-checklist.html" rel="nofollow">contributor checklist</a>.</p>
<p>Note that you can go back and touch up your existing commits with <code>git rebase --interactive</code>. You don't strictly need to do this, but if you're not already familiar with this hugely powerful Git tool, I'd recommend taking a little quality time to learn it. It allows you to rewrite history, change details about commits, etc. In this case, what you want to do is "reword" the commits above to conform to the style guidelines. Again, not a strict requirement at all, but mentioning it as this is a tool I use every day, and usually find that others are very happy to know about once they've learned it too.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq-docs/pull/43#issuecomment-385974667">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtr8Mp2yb_fiC8c8lA_Xl6cjIVTM9ks5tubJMgaJpZM4TvXHe">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtlpZSqRcNHYbM9VFwEYVCbo99C81ks5tubJMgaJpZM4TvXHe.gif" height="1" width="1" alt="" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/bisq-network/bisq-docs/pull/43#issuecomment-385974667"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq-docs","title":"bisq-network/bisq-docs","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq-docs"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cbeams in #43: Looks great at a glance, @m52go, I'll provide more detailed comments later, but in the meantime, I've pushed a couple review commits, please see and study them above.\r\n\r\nA few other nits:\r\n\r\n 1. Pease read and follow our guidelines around commit comments at bisq-network/style#9.\r\n 2. Configure your Git client for proper author metadata (first and last name)\r\n 3. Set yourself up for GPG commit signing\r\n 4. In the future, please submit PRs from a branch other than your `master` branch, e.g. in this case, `getting-started` would have been fine. I'll omit the reasons for this here, but can explain more fully later if you like.\r\n\r\nLinks to and further information about each of these things can be found in the [contributor checklist](https://docs.bisq.network/contributor-checklist.html).\r\n\r\nNote that you can go back and touch up your existing commits with `git rebase --interactive`. You don't strictly need to do this, but if you're not already familiar with this hugely powerful Git tool, I'd recommend taking a little quality time to learn it. It allows you to rewrite history, change details about commits, etc. In this case, what you want to do is \"reword\" the commits above to conform to the style guidelines. Again, not a strict requirement at all, but mentioning it as this is a tool I use every day, and usually find that others are very happy to know about once they've learned it too."}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-docs/pull/43#issuecomment-385974667"}}}</script>