[PATCH] pullrequests: add support for custom pull request id prefix
Mads Kiilerich
mads at kiilerich.com
Mon Apr 20 17:41:35 EDT 2015
On 04/20/2015 06:02 AM, Thomas De Schampheleire wrote:
> On Sun, Apr 19, 2015 at 6:42 PM, Mads Kiilerich <mads at kiilerich.com> wrote:
>> By the way ... it has been mentioned before that the term "pull request" is
>> weird for most workflows.
>>
>> Inspired by discussions with people at the Mercurial sprint, I think I am
>> settling on using the term "series". It is all about presenting a series of
>> changes / patches / changesets so they can be previewed, reviewed, commented
>> on, approved or rejected, pulled and tested, rebased, grafted, "evolved" and
>> whatever you can come up with.
> How would this term 'series' be used?
> Now you can 'create a pull request' (which as Jan mentioned we have
> translated as 'create a review request'. Would this become 'create a
> series' ?
You really start creating the series when you start creating local
commits that don't get pushed to the upstream mainline immediately. It
becomes more necessary to have them as a manageable concept when you
start having multiple such series locally or want to share them with
others for collaboration, review or upstreaming.
So it will be more like "put this series up for review", "update the
series according to reviews", "review this series", "I applied your
series to the mainline".
Just an idea, but I like it.
> You mentioned inspiration from the Mercurial sprint: how is this term
> used in Mercurial? I'm aware of mq series, but haven't seen the term
> elsewhere.
It is not directly used in Mercurial the tool or project but in the work
flows around Mercurial when using other tools like ReviewBoard or
Phabricator. From my point of view, evolve is all about reinventing mq
and thus they also need a "qseries" concept ;-)
/Mads
More information about the kallithea-general
mailing list