mediagoblin-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [GMG-Devel] Pull requests on Gitorious


From: Rj Ewing
Subject: Re: [GMG-Devel] Pull requests on Gitorious
Date: Fri, 17 May 2013 12:29:16 -0700

I was looking through some of the merge requests, and it looks like a lot of them, especially the older ones, are already merged.

Ex.
http://gitorious.org/mediagoblin/mediagoblin/merge_requests/23 
http://bugs.foocorp.net/issues/444#note-7


On Fri, May 17, 2013 at 10:53 AM, Christopher Allan Webber <address@hidden> wrote:
That would work great.

Jim Campbell writes:

> 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
>>

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


reply via email to

[Prev in Thread] Current Thread [Next in Thread]