I merged this -- but it's not written in stone! Please feel free to bring up any improvements in any of our processes, of course!
Ryan On Fri, Sep 2, 2022 at 6:17 PM Oscar Westra van Holthe - Kind <[email protected]> wrote: > > Thank you Ryan! This version looks much more friendly. > > > Kind regards, > Oscar > > -- > Oscar Westra van Holthe - Kind <[email protected]> > > Op do 1 sep. 2022 22:48 schreef Martin Grigorov <[email protected]>: > > > Hi, > > > > +1 for the improvements! > > > > I hope that less people will ignore it now after explicitly stating that if > > the template is ignored then the review will be postponed :-) > > > > Martin > > > > On Thu, Sep 1, 2022 at 10:04 PM Ryan Skraba <[email protected]> wrote: > > > > > Hey everybody -- > > > > > > For the most part, it seems that the PR template that we use is > > > largely ignored, and the checklist is seldom actually applied! Github > > > reports on the number of tasks done in the checklist, but most merged > > > PRs don't have 4 tasks out of 4 completed. It's kind of misleading. > > > > > > I think we can probably do better and have more descriptive PRs with > > > less general boilerplate to encourage more targeted text! > > > > > > I put up a proposal shamelessly adapted from Flink at > > > https://github.com/apache/avro/pull/1851, all comments are welcome! > > > > > > All my best, Ryan > > > > >
