Hi Jimmy, On Tue, May 12, 2015 at 10:39 AM, <dev-digest-h...@climate.apache.org> wrote:
> Since this has been brought up a few times on various tickets I thought now > would be a good time to go over our project workflow and make sure it's > working for us. > Excellent idea! > > A general overview of the workflow that we use is available at [1]. A brief > overview is that: > - ALL changes are pushed up to Github for code review before being merged. > - If no issues are raised within a reasonable amount of time (usually 72 > hours is what I stick with) those changes can be merged. > This seems very reasonable to me. It has been stable for a good while. There are some 26 people with write permissions to the repository. a 72 hour grace period for pull requests seems very logical and very permissive to me. Pull requests are very well understood. If people are not familiar with pull requests and with Guthub workflows then they are always welcome to come to the ML. Lewis
