
Simon, your suggestion makes sense to me. We can have 3 pages that cover: 1. Fixing Bugs (/working-conventions/fixing-bugs) 2. Adding Features (/working-conventions/adding-features) 3. Merge request work-flow (New page e.g. (/working-conventions/merge-requests) Then (1) and (2) would reference (3) and most of the content in /home would be moved to (3). This leave one loose end: /home is left almost empty. I suggest we move contents of /contributing https://gitlab.haskell.org/ghc/ghc/wikis/contributing to /home, though I'm interested in hearing any alternative idea - David E On 5/23/19 12:24 PM, Simon Peyton Jones wrote:
Thanks
* I've noted this infixing-bugs https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fworking-conventions%2Ffixing-bugs&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062540850&sdata=3fwFLSB2V31MvPzaX8inpYgKVq4%2FOFaA8Vdv7oIuxW4%3D&reserved=0 and linked tohome https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fhome&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062540850&sdata=qZXL1kW4dnqeX%2BMY0IV3KzNu%2F0eovLt7bayC2tUKUkA%3D&reserved=0 (I'm trying to avoid too much duplication betweenhttps://gitlab.haskell.org/ghc/ghc/wikis/working-conventions/fixing-bugs https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fworking-conventions%2Ffixing-bugs&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062550846&sdata=1KsjtG1qLyLuHmORwiFUu%2FSjuAH7bfkoMsBRzJx0IEc%3D&reserved=0 andhttps://gitlab.haskell.org/ghc/ghc/wikis/home https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fhome&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062550846&sdata=%2B8hui7k5Wa9lLwTji0mjnfNMDNDXL%2BZIk9ANS2BCkus%3D&reserved=0)
I agree with avoiding duplication, but let’s fix that by fixing the home page. Why is there all this detail about merge requests there? Let’s push all that off into a single integrated page “How to contribute a patch”.
The current “how to contribute a patch” page is really “how to fix a bug (including how to contribute a patch)”. So maybe we should have:
* How to fix a bug * How to add a feature
both as pages of their own, but both pointing to a single pate
* The mechanics of contributing a patch (bug or feature)
That would also de-clutter the home page.
Simon
*From:*David Eichmann
*Sent:* 23 May 2019 12:10 *To:* Simon Peyton Jones ; Ben Gamari *Cc:* iavor.diatchki@gmail.com *Subject:* Re: Processing MRs very slow? I found the email you're referring to https://mail.haskell.org/pipermail/ghc-devs/2019-May/017656.html https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail.haskell.org%2Fpipermail%2Fghc-devs%2F2019-May%2F017656.html&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062530854&sdata=fYB%2F%2B46qTB%2FqtBZXNZSZ2s1nCp4K0LM7zLNo3GpSRvU%3D&reserved=0. You mentioned 3 points of missing information:
* Picking approvers * Lets see how the "gitlab upgrade - change in approval system" email thread develops. * Assigning to Marge * I've noted this infixing-bugs https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fworking-conventions%2Ffixing-bugs&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062540850&sdata=3fwFLSB2V31MvPzaX8inpYgKVq4%2FOFaA8Vdv7oIuxW4%3D&reserved=0 and linked tohome https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fhome&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062540850&sdata=qZXL1kW4dnqeX%2BMY0IV3KzNu%2F0eovLt7bayC2tUKUkA%3D&reserved=0 (I'm trying to avoid too much duplication betweenhttps://gitlab.haskell.org/ghc/ghc/wikis/working-conventions/fixing-bugs https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fworking-conventions%2Ffixing-bugs&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062550846&sdata=1KsjtG1qLyLuHmORwiFUu%2FSjuAH7bfkoMsBRzJx0IEc%3D&reserved=0 andhttps://gitlab.haskell.org/ghc/ghc/wikis/home https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fhome&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062550846&sdata=%2B8hui7k5Wa9lLwTji0mjnfNMDNDXL%2BZIk9ANS2BCkus%3D&reserved=0) * Monitoring progress if it doesn't land within 24 hrs; even knowing when to time out would help. * I've added some info inhttps://gitlab.haskell.org/ghc/ghc/wikis/home#merging-your-merge-request https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fhome%23merging-your-merge-request&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062560836&sdata=PfpXM9lemGz9NgH3B9mhSmWwUsAr9QDSTf0dMqXFK1I%3D&reserved=0 Hope this is better now. Ben maybe you'd like to double check the bullet list inhttps://gitlab.haskell.org/ghc/ghc/wikis/home#merging-your-merge-request https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.haskell.org%2Fghc%2Fghc%2Fwikis%2Fhome%23merging-your-merge-request&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062560836&sdata=PfpXM9lemGz9NgH3B9mhSmWwUsAr9QDSTf0dMqXFK1I%3D&reserved=0 for the conditions under which marge will batch your MR. - David E
On 5/23/19 9:02 AM, Simon Peyton Jones wrote:
| | Perhaps it is worth updating Step 7 or 8 of
Yes, I suggested that to Ben a week or two ago, but he's flat out on incremental GC.
Maybe David could do it? I can dig out the email.
Thanks!
Simon
| -----Original Message-----
| From: ghc-devs
mailto:ghc-devs-bounces@haskell.org On Behalf Of Iavor Diatchki | Sent: 22 May 2019 23:49
| To: David Eichmann
mailto:davide@well-typed.com | Cc: ghc-devs
mailto:ghc-devs@haskell.org | Subject: Re: Processing MRs very slow?
|
| Great thanks! How can I have a look at Marge-bot's queue?
|
| Perhaps it is worth updating Step 7 or 8 of
|https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.h
| askell.org%2Fghc%2Fghc%2Fwikis%2Fworking-conventions%2Ffixing-
| bugs&data=02%7C01%7Csimonpj%40microsoft.com%7Cc0e7235773dd4c553d8c08d6
| df07cbb3%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636941621907873036&a
| mp;sdata=gWoMbrHZwyfDQWeCst0VmAl4x%2Fv%2BjFY3dUXI7X%2FCHEQ%3D&reserved
| =0
| to indicate that the reviewers should assign the MR to Marge-bot when it
| is approved, and how one can check if that has happened.
|
|
| On Wed, May 22, 2019 at 9:53 AM David Eichmann
mailto:davide@well-typed.com | wrote:
| >
| > Recently there have been some issues with CI that has slowed down
| > merging. In particular a performance test (T9630) was failing on CI.
| > That is fixed now and as of today Marge-Bot seems to be merging MRs
| > again. I'll continue to monitor this in the coming days.
| >
| > Iavor, your MR was approved but not assigned to Marge-bot, and so was
| > not in the merge queue. I'm not sure who has permission to do this,
| > but you can always ping the approvers. In this case I've assigned to
| > Marge-bot for you, and it will hopefully be merge soon.
| >
| > David Eichmann
| >
| > On 5/22/19 5:09 PM, Iavor Diatchki wrote:
| > > Hello,
| > >
| > > I made a gitlab MR that adds two NOINLINE pragmas and some comments.
| > > It's been about a week since my last push to the MR, and nothing has
| > > happened since.
| > >
| > > Is there a way to check on what is its status:
| > > - Is it stuck because I need to do something?
| > > - Is it stuck because someone else needs to do something?
| > > - Or is it just in the queue to be merged, in which case it would
| > > be nice to know where in line it is, so I can see that there is
| > > progress, and it is not just stuck.
| > >
| > > Given that this is such a simple MR, I am not too worried about
| > > conflicts but a week long lag seems less than ideal for anything
| > > even mildly complex.
| > >
| > > -Iavor
| > >
| > > PS: I just clicked on all the little check boxes from the
| > > template---perhaps that's why things weren't progressing?
| > > _______________________________________________
| > > ghc-devs mailing list
| > >ghc-devs@haskell.org mailto:ghc-devs@haskell.org
| > >https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmai
| > > l.haskell.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fghc-devs&data=02%
| > > 7C01%7Csimonpj%40microsoft.com%7Cc0e7235773dd4c553d8c08d6df07cbb3%7C
| > > 72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636941621907873036&sd
| > > ata=W3wnIH%2F2oEmuqona%2Fk6NtoYUZsnFA1kun%2Blm%2BSGMIjo%3D&reser
| > > ved=0
| >
| > --
| > David Eichmann, Haskell Consultant
| > Well-Typed LLP,
| >https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.w
| > ell-typed.com&data=02%7C01%7Csimonpj%40microsoft.com%7Cc0e7235773d
| > d4c553d8c08d6df07cbb3%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636
| > 941621907873036&sdata=RIcEblo4xNED7UY6Vv%2FNIG4aAzcjLgzN8uegqX9K%2
| > F3I%3D&reserved=0
| >
| > Registered in England & Wales, OC335890
| > 118 Wymering Mansions, Wymering Road, London W9 2NF, England
| >
| > _______________________________________________
| > ghc-devs mailing list
| >ghc-devs@haskell.org mailto:ghc-devs@haskell.org
| >https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmail.
| > haskell.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fghc-devs&data=02%7C01
| > %7Csimonpj%40microsoft.com%7Cc0e7235773dd4c553d8c08d6df07cbb3%7C72f988
| > bf86f141af91ab2d7cd011db47%7C1%7C0%7C636941621907873036&sdata=W3wn
| > IH%2F2oEmuqona%2Fk6NtoYUZsnFA1kun%2Blm%2BSGMIjo%3D&reserved=0
| _______________________________________________
| ghc-devs mailing list
|ghc-devs@haskell.org mailto:ghc-devs@haskell.org
|https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmail.hask
| ell.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fghc-
| devs&data=02%7C01%7Csimonpj%40microsoft.com%7Cc0e7235773dd4c553d8c08d6
| df07cbb3%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636941621907873036&a
| mp;sdata=W3wnIH%2F2oEmuqona%2Fk6NtoYUZsnFA1kun%2Blm%2BSGMIjo%3D&reserv
| ed=0
-- David Eichmann, Haskell Consultant Well-Typed LLP,http://www.well-typed.com https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.well-typed.com&data=02%7C01%7Csimonpj%40microsoft.com%7Ca1d7bf2f0b3a498d6e2508d6df6f346c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636942066062570831&sdata=HFFSJocB9Dyvnf1THqbOGkQFfFVUJbzKOMRwZlvxfLw%3D&reserved=0 Registered in England & Wales, OC335890 118 Wymering Mansions, Wymering Road, London W9 2NF, England
-- David Eichmann, Haskell Consultant Well-Typed LLP, http://www.well-typed.com Registered in England & Wales, OC335890 118 Wymering Mansions, Wymering Road, London W9 2NF, England

This leave one loose end: /home is left almost empty
I think /home can give a little overview of what’s on the wiki, including:
* A copy of what’s in the side-bar, annotated with what’s in each bit.
* A pointer to the title list for the wiki (which we auto-generate I think).
I’m sure other stuff will occur to us!
Thanks for doing this.
Simon
From: David Eichmann

David
Ben writes (on #16586)
@simonpj</simonpj>, in general GHC HQ sets the milestone during triage (which I did in this case; I only realized that Iavor had submitted an MR after I left the comment requesting that he set the milestone).
Could you include this point in your workflow revisions?
Simon
From: David Eichmann
participants (2)
-
David Eichmann
-
Simon Peyton Jones