Minor doc patch for ghc 7.10 + did I submit the patch correctly/follow the process?

Doug Burke dburke.gw at gmail.com
Tue Mar 17 23:45:18 UTC 2015


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.

Doug

On Tue, Mar 17, 2015 at 7:04 PM, Thomas Miedema <thomasmiedema at gmail.com>
wrote:

> Hi Doug,
>
> thank you for describing the steps you took to submit a code review.
> That's quite useful. Your patch looks good too.
>
> > b) ask if I did this the right way; I'm happy to write this up as an
> > addendum to the newcomers guide/something else on the wiki if it is a
> > sensible path for a doc-only patch
>
> Those steps you mention seem about right. Only this would have gotten
> you there somewhat sooner:
>
> https://ghc.haskell.org/trac/ghc/wiki/Building/GettingTheSources#GettingabranchGHC7.9orlater
>
> I don't think that the Newcomers page should have to mention that
> specific link however, because:
> 1. almost all patches are made for the master branch. That release
> notes file could be the only exception.
> 2. the wiki contains a lot of information that would be useful to read
> for newcomers, but we can't link to all of them
> 3. there is already a link to
> https://ghc.haskell.org/trac/ghc/wiki/Building/GettingTheSources
> 4. you proved you could make it happen without it
>
> Other things you mentioned:
> > 1) I did not create a ticket; should I have done so?
>
> Just a Phabricator code review is ok for small patches. Since we
> started using Phabricator, quite a few patches have been submitted and
> accepted without a ticket number. This workflow is however not
> mentioned on the wiki yet.
>
> > downloading the submodules seems excessive for a doc-only patch
> Maybe, but it's also more simple to just have one explanation for how
> to get the sources instead of two. But if you think the Newcomers page
> could be improved, please feel free to edit it.
>
> -Thomas
>
> P.S. The Newcomers page makes you download from Github to prevent
> git.haskell.org from becomming overloaded when the Newcomers page gets
> linked from reddit.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/ghc-devs/attachments/20150317/161ae50c/attachment.html>


More information about the ghc-devs mailing list