|
From: | Jim Campbell |
Subject: | Re: [GMG-Devel] Pull requests on Gitorious |
Date: | Tue, 21 May 2013 19:58:47 -0500 |
Here's some more:
Don't have a link to the ticket, but the changes are in the current code
That's all for now, but if Jim goes reviews the doc related merge requests, there should only be a couple left.On Fri, May 17, 2013 at 3:27 PM, Christopher Allan Webber <address@hidden> wrote:
Thanks for the heads up; closed! Help like this in cleaning up the list
is appreciated! :)
Rj Ewing writes:
> 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
>>
[Prev in Thread] | Current Thread | [Next in Thread] |