[ckan-dev] Merging of pull requests

Tue Oct 15 16:46:51 UTC 2013

Hi Stefan,

Your questions are totally understandable and I'm sure also shared by
other developers, we are trying to be more transparent about the CKAN
development process so any feedback is appreciated.

We triage any new PR and issues two times a week, on Tuesday and
Thursday. Each of them gets assigned to a CKAN core dev, which will
oversee it. Most of the times this involves working on a patch or
reviewing it, but also asking the original submitter for more details
or even reject it if it does not fit the guidelines. If one of your
submissions does not get assigned to a dev after week leave a comment
or ping the list as it may have slipped us.

As for when things will get done, that is a more difficult question to
answer, it depends on the importance of the patch, other priorities or
general availability, specially on extensions where fewer devs are
responsible for its maintenance.

Hope this helps,

Adrià

On 15 October 2013 15:12, Stefan Oderbolz <stefan.oderbolz at liip.ch> wrote:
> Hi there,
>
> I was wondering how you handle the review and merge of pull requests. Is
> there some kind of protocol to follow (like ping person X) or is there a
> schedule when you go though open PRs (like every second Tuesday we tackle
> all open PRs)?
>
> Just to make sure that you don't get me wrong: this mail is *not* about
> complaining, I just want to know how you handle that internally (in OKFN),
> who is responsible for the ckan core or for the different ckanext-* repos.
> If there are no such "rules" I'd propose to create them, this helps me (and
> probably others in the community) to understand how you work and what we can
> expect when.
>
> Thanks in advance for any hints :)
>
> - Stefan
>
> --
> Liip AG  //  Feldstrasse 133 //  CH-8004 Zurich
> Tel +41 43 500 39 80 // GnuPG 0x7B588C67 // www.liip.ch
>
> _______________________________________________
> ckan-dev mailing list
> ckan-dev at lists.okfn.org
> http://lists.okfn.org/mailman/listinfo/ckan-dev
> Unsubscribe: http://lists.okfn.org/mailman/options/ckan-dev
>