Re: [GMG-Devel] Pull requests on Gitorious

[Jim Campbell]

I can review some of the documentation-related merge requests. Can get to them this weekend.

It looks like some of them may be going over some of the same sets of files. I can try to merge them together, fix any conflicts, and then re-submit upstream for review . . . would that work? I don't really see a pretty way of handling things otherwise.

Jim

On Tue, May 14, 2013 at 9:25 AM, Christopher Allan Webber <address@hidden> wrote:

Heya all,

So you're right in observing that merge requests in gitorious get mostly
ignored.

I created an issue about this not too long ago:
  http://issues.mediagoblin.org/ticket/693

Basically, merge requests aren't within our normal workflow of feature
adding, which is described on our wiki:
  http://wiki.mediagoblin.org/Git_workflow

This means I'm often failing to pay attention to merge requests.  The
ticket I posted above has the goal of removing merge requests and
migrating the old ones.  Nonetheless, I've put both of your merge
requests onto my "review queue" so I'll try to get to them soon.

I could use help with that above ticket, though.  Triaging all those old
merge requests is going to be... a lot of work. :)  If anyone thinks
they can help either by assisting with review or moving them over to the
issue tracker, I'd appreciate it.

 - Chris

Benjamin Lebsanft writes:

> #54 would be nice too, although no need to use 100 for the jpeg
> compression settings.
>
> Bye
> Ben
>
> Am 14.05.2013 13:48, schrieb Sam Tuke:
>> Hi all,
>>
>> I submitted a pull request for a very simple documentation-related
>> commit a week ago[1]. I'm just wondering if anyone is monitoring pull
>> requests to the repo? Seems like there are quite a few waiting pull
>> requests from others with no comments from the core team[2].
>>
>> Best,
>>
>> Sam.
>>
>> 1. https://gitorious.org/mediagoblin/mediagoblin/merge_requests/58
>> 2. https://gitorious.org/mediagoblin/mediagoblin/merge_requests
>> _______________________________________________
>> devel mailing list
>> address@hidden
>> http://lists.mediagoblin.org/listinfo/devel
>>
>
> _______________________________________________
> devel mailing list
> address@hidden
> http://lists.mediagoblin.org/listinfo/devel

_______________________________________________
devel mailing list
address@hidden
http://lists.mediagoblin.org/listinfo/devel