<div dir="ltr"><div><div><br></div>Thanks for the confirmation. There's always a tension between simplicity and completeness, but I'll have a look and see if I can come up with some suggestions.<br><br></div>Doug<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 17, 2015 at 7:04 PM, Thomas Miedema <span dir="ltr"><<a href="mailto:thomasmiedema@gmail.com" target="_blank">thomasmiedema@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Doug,<br>
<br>
thank you for describing the steps you took to submit a code review.<br>
That's quite useful. Your patch looks good too.<br>
<span class=""><br>
> b) ask if I did this the right way; I'm happy to write this up as an<br>
> addendum to the newcomers guide/something else on the wiki if it is a<br>
> sensible path for a doc-only patch<br>
<br>
</span>Those steps you mention seem about right. Only this would have gotten<br>
you there somewhat sooner:<br>
<a href="https://ghc.haskell.org/trac/ghc/wiki/Building/GettingTheSources#GettingabranchGHC7.9orlater" target="_blank">https://ghc.haskell.org/trac/ghc/wiki/Building/GettingTheSources#GettingabranchGHC7.9orlater</a><br>
<br>
I don't think that the Newcomers page should have to mention that<br>
specific link however, because:<br>
1. almost all patches are made for the master branch. That release<br>
notes file could be the only exception.<br>
2. the wiki contains a lot of information that would be useful to read<br>
for newcomers, but we can't link to all of them<br>
3. there is already a link to<br>
<a href="https://ghc.haskell.org/trac/ghc/wiki/Building/GettingTheSources" target="_blank">https://ghc.haskell.org/trac/ghc/wiki/Building/GettingTheSources</a><br>
4. you proved you could make it happen without it<br>
<br>
Other things you mentioned:<br>
<span class="">> 1) I did not create a ticket; should I have done so?<br>
<br>
</span>Just a Phabricator code review is ok for small patches. Since we<br>
started using Phabricator, quite a few patches have been submitted and<br>
accepted without a ticket number. This workflow is however not<br>
mentioned on the wiki yet.<br>
<span class=""><br>
> downloading the submodules seems excessive for a doc-only patch<br>
</span>Maybe, but it's also more simple to just have one explanation for how<br>
to get the sources instead of two. But if you think the Newcomers page<br>
could be improved, please feel free to edit it.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Thomas<br>
</font></span><br>
P.S. The Newcomers page makes you download from Github to prevent<br>
<a href="http://git.haskell.org" target="_blank">git.haskell.org</a> from becomming overloaded when the Newcomers page gets<br>
linked from reddit.<br>
</blockquote></div><br></div>