hi jun,
I have concerned the inappropriate of merging one's pr by one's self. I will not do it like this. About this pr, welcome to review it. If it's wrong, please tell me to fix it or would you please to fix it. Thank you! Taosheng ------------------ Original ------------------ From: jun liu <ken.lj...@gmail.com> Date: Thu,Mar 28,2019 2:26 PM To: dev@dubbo.apache.org <dev@dubbo.apache.org> Subject: Re: never create branch on main repo Hi, Taosheng For the exact PR[1] that is related to this accident branch, I think it should be reconsidered, I am not sure if the changes in it are right, I will take a look at it later. I noticed you merged it by yourself, since there is no sign suggests that it??s an emergency patch or in some situation worthing that, I would recommend wait for other committers to review and to decide when to merge it. I remember Huxing has a thread discussing the collaborating principles, please feel free to express your concerns on this. 1. https://github.com/apache/incubator-dubbo/pull/3688 Jun > On Mar 28, 2019, at 1:20 AM, Taosheng, Wei <weitaosh...@foxmail.com> wrote: > > Hi, > I have deleted the branch. And I remember why this happened. At the > beginning, I tried to revert the pr 3688, so I click the "revert" button of > github. And the github should created a branch on the main repo, but I didn't > find it. And then I didn't want to revert the pr, so I closed the revert pr. > But the branch created by github automatically was remained. > So I just want to remind that we should be careful when we need to revert one > pr. Remembering that the revert function provided by github will create a > branch on the main repo. > > > > > ------------------ Original ------------------ > From: "Taosheng, Wei"<weitaosh...@foxmail.com>; > Date: Wed, Mar 27, 2019 11:03 PM > To: "dev"<dev@dubbo.apache.org>;"dev"<dev@dubbo.apache.org>; > > Subject: Re: never create branch on main repo > > > > hi Ian, > I am so sorry to make the mistake. I just wanted to submit a pr, but it make > a mistaken operation of creating a branch. I will be more careful when trying > to submit a pr. > > > ------------------ Original ------------------ > From: Ian Luo <ian....@gmail.com> > Date: Tue,Mar 26,2019 5:45 PM > To: dev <dev@dubbo.apache.org> > Subject: Re: never create branch on main repo > > > > Hi Folks, > > Recently I noticed there's a branch named > 'revert-3688-config-center-config' created on the main repo. This is > unacceptable even if you are granted with the committer privilege. > > Pls. **ALWAYS** go through pull request unless you are working on release. > > Thanks, > -Ian.