+1 Sorry Nicolò Boschi, we missed your email.
Best, Penghui On Thu, Aug 18, 2022 at 10:00 PM Nicolò Boschi <boschi1...@gmail.com> wrote: > +1. Actually I suggested the same a month ago > https://lists.apache.org/thread/p3c6pb6gfgdb5mmdjky6h7njkt8ggl2v > > Nicolò Boschi > > > Il giorno gio 18 ago 2022 alle ore 15:54 Yunze Xu > <y...@streamnative.io.invalid> ha scritto: > > > I’m glad to see the “update branch” option enabled for contributors. > > > > Thanks, > > Yunze > > > > > > > > > > > 2022年8月18日 21:01，tison <wander4...@gmail.com> 写道： > > > > > > Hello, > > > > > > The short version > > > ============= > > > > > > Vote if you agree on enabling the non-mandatory updating PR branches > > > button, i.e., the "Always suggest updating pull request branches" > GitHub > > > settings. > > > > > > The full version > > > ============ > > > > > > Pulsar is under rapid development and numerous fixes are pushed to > master > > > every time. Since we are still suffering from quite a few flaky tests, > > > merge master and retest is a hotspot to verify the patch once more. > > > > > > However, we should pull the nightly master locally, check out the PR > > > branch, perform the merge and push to remote. It's a bit awkward > > especially > > > when a developer works on multiple branches. > > > > > > GitHub provides a button "Always suggest updating pull request > branches" > > > with the description "Whenever there are new changes available in the > > base > > > branch, present an “update branch” option in the pull request."[1] > > > > > > It can simplify the workflow with one button click. > > > > > > To clarify, this is different from the branch protection rule "Require > > > branches to be up to date before merging" - it's non-mandatory and just > > > provides the "update branch" button. It means we don't force every PR > to > > > catch up with the latest master before merged, which can cause > > exextremely > > > high unnecessary traffic. Since we already allow PR authors to > retrigger > > > tests with the pulsorbot (or even contributors can push an empty > commit), > > > providing such a button does no harm. > > > > > > I post this thread here to collect feedback, especially from the PMC > > > members. Previously I asked the INFRA team to turn on this option for > > > Apache Kvrocks (Incubating)[2] and I believe the INFRA team would be > > happy > > > to see an explicit community consensus. > > > > > > Best, > > > tison. > > > > > > [1] > > > > > > https://github.blog/changelog/2022-02-03-more-ways-to-keep-your-pull-request-branch-up-to-date/ > > > [2] https://issues.apache.org/jira/browse/INFRA-23432 > > > > >
