On Mon, Jul 11, 2016 at 10:26 AM, Aki Sukegawa <ns...@apache.org> wrote:
> Jim, most of them are submitted after initial planned date of 0.10.0 > release that is January. > I believe it is more important to proceed with the next release right now. > That said, I agree we need to work through pull requests. > I've written while ago a small tool to list all pull requests which have > closed JIRA tickets or no ticket, so maybe we can ask Apache INFRA team (or > maybe directly Jake ?) to close them at once later. > > > John, FWIW CI works for the current master, so it's more about how it > looks. > I understand bad looking CI result = less contributors and it needs to be > fixed eventually, but I don't think we should delay the release even > further for this. > I am not currently contributing, so my opinion is just that, but a green CI, which is the visible one for master (and thus for pull requests by inference), is what I was hoping 0.10.0 would block for. If master CI is red 75% of the time, a good PR will also be red 75% of the time and there will be no solid way to identify bad PRs. This should lead quickly to red master. > About the contributing advice, which part you find outdated ? > Maybe we can remove JIRA-patch part, as it seems that many of us are liking > Github PR. > The JIRA patch did seem possibly outdated to me. I more wanted o call attention to the contributor experience as a whole. Contributors will read that page (hopefully!), so I was just trying to prompt a review of it. > > On Tue, Jul 12, 2016 at 1:02 AM John Sirois <john.sir...@gmail.com> wrote: > > > I'm excited to see progress towards a sane state for contributors to pile > > on, but, in addition to Jim's point, I have a few concerns: > > + Which build is the pre-commit? The master CI looks ~75% red: > > https://travis-ci.org/apache/thrift/builds - still dauntingly red for a > > new > > contributor imo. > > + Is the contributing advice still accurate?: > > https://thrift.apache.org/docs/HowToContribute > > > > On Mon, Jul 11, 2016 at 9:47 AM, Jim King <jim.k...@simplivity.com> > wrote: > > > > > I suggested to Jake in an email that we should drain the pull request > > list > > > before we do this. There are 110+ open pull requests on github. That > > size > > > backlog is not healthy. Anything recent that can be rebased and merged > > > should be included. Anything older than a couple months should > probably > > be > > > declined outright and the author may resubmit it. > > > > > > - Jim > > > > > > -----Original Message----- > > > From: Aki Sukegawa [mailto:ns...@apache.org] > > > Sent: Monday, July 11, 2016 5:14 AM > > > To: dev@thrift.apache.org > > > Subject: Re: 0.10.0 > > > > > > +1 ! > > > > > > On Mon, Jul 11, 2016, 17:07 Jens Geyer <jensge...@hotmail.com> wrote: > > > > > > > +1 > > > > ________________________________ > > > > Von: Jake Farrell > > > > Gesendet: 11.07.2016 04:02 > > > > An: dev@thrift.apache.org > > > > Betreff: 0.10.0 > > > > > > > > With the builds now solely on travis and the pre-commit green again > > > > what are peoples thoughts on cutting 0.10.0, any objections? > > > > > > > > -Jake > > > > > > > > > > > > > --------------------------------------------------------------------------------------- > > > PRIVACY STATEMENT: > > > This message is a PRIVATE communication. This message and all > > attachments > > > are a private communication sent by SimpliVity and are considered to be > > > confidential or protected by privilege. If you are not the intended > > > recipient, you are hereby notified that any disclosure, copying, > > > distribution or use of the information contained in or attached to this > > > message is strictly prohibited. Please notify the sender of the > delivery > > > error by replying to this message, and then delete it from your system. > > > > > > > > > --------------------------------------------------------------------------------------- > > > > > >