From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2B261C4360C for ; Sun, 29 Sep 2019 11:57:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id F12B720863 for ; Sun, 29 Sep 2019 11:57:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725937AbfI2L5i (ORCPT ); Sun, 29 Sep 2019 07:57:38 -0400 Received: from charlotte.tuxdriver.com ([70.61.120.58]:51149 "EHLO smtp.tuxdriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725924AbfI2L5i (ORCPT ); Sun, 29 Sep 2019 07:57:38 -0400 Received: from cpe-2606-a000-111b-43ee-0-0-0-115f.dyn6.twc.com ([2606:a000:111b:43ee::115f] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1iEXpP-0005Hf-G2; Sun, 29 Sep 2019 07:57:31 -0400 Date: Sun, 29 Sep 2019 07:57:22 -0400 From: Neil Horman To: Steven Rostedt Cc: Laurent Pinchart , Drew DeVault , workflows@vger.kernel.org Subject: Re: thoughts on a Merge Request based development workflow Message-ID: <20190929115722.GA26820@hmswarspite.think-freely.org> References: <20190924182536.GC6041@hmswarspite.think-freely.org> <20190924185312.GD6041@hmswarspite.think-freely.org> <20190924202423.GA14425@pendragon.ideasonboard.com> <20190924222502.GA11633@hmswarspite.think-freely.org> <20190925205036.GA7763@pendragon.ideasonboard.com> <20190926004045.GA20302@localhost.localdomain> <20190928185848.76c85a9d@oasis.local.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190928185848.76c85a9d@oasis.local.home> User-Agent: Mutt/1.12.1 (2019-06-15) Sender: workflows-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: workflows@vger.kernel.org On Sat, Sep 28, 2019 at 06:58:48PM -0400, Steven Rostedt wrote: > On Wed, 25 Sep 2019 20:40:45 -0400 > Neil Horman wrote: > > > Eventually, barring any really significant objection, hes going to make > > the switch, and users will either have to get github accounts, or stop > > participating in netdev development. > > That will be a very sad day if that happened. > > Whatever service should have an email interface. For example, if I get > a message from bugzilla.kernel.org, I can reply back via email and it > is inserted into the tool (as I see my Out of office messages going > into it. I need to fix my scripts not to reply to bugzilla). > Forge solutions do have the ability to use email as an interface to issue tracking, thats not a problem. What they don't currently seem to have is the ability to emulate patch review workflows. And thats not to say they couldn't, but it seems to me that they haven't prioritized that because they offer several different types of comment options (commenting in the pull request discussion(s) themselves vs commenting on code, etc. If they sould implement that, I think alot of this would become alot easier. > I set up patchwork on my INBOX, as I'm having a hard time of separating > patches from the noise. And it works really well. I would love to be > able to push my patchwork list to a public place so that others can see > it too. As mentioned in the Maintainers Summit, it would be great to be > able to pull patchwork down to my laptop, get on the plane, process a > bunch of patches while flying, and then when I land, I could push the > updates to the public server. > > That's pretty much all I'm looking for. > I think what you are looking for here is a way to pull down a set of merge requests, review and merge those you approve, and push them back when you are back online? I think you can do at least some of that. Forge solutions (definately gitlab, likely github), allow you to pull a merge request reference namespace (on gitlab its heads/merge_requests/). You can merge whatever head there you like to its intended target branch, and when you push, it will update the corresponding MR to the MERGED state. What you can't currently do is make a comment on an MR, store that comment in git and then have the MR updated with those comments. That would be a great item to make that feature more complete. Neil > -- Steve >