On Tue, Jun 19, 2018 at 10:54 PM, Huxing Zhang <[email protected]> wrote: > On Tue, Jun 19, 2018 at 6:11 PM, Xin Wang <[email protected]> wrote: >> https://travis-ci.org/apache/incubator-dubbo-ops >> https://travis-ci.org/apache/incubator-dubbo-spring-boot-project >> >> Now the two projects were not activated in travis-ci.org >> >> Only the owner of incubator-dubbo-ops and >> incubator-dubbo-spring-boot-project can activate them >> >> How do you deal with it below? Create a issue on JIRA to handle it? >> @[email protected] > > I have create an Infra ticket: > https://issues.apache.org/jira/browse/INFRA-16663
According to the the infra ticket, it should be enabled. > >> >> Yong Zhu <[email protected]> 于2018年6月19日周二 下午5:54写道： >>> >>> On Mon, Jun 18, 2018 at 5:13 AM Mark Thomas <[email protected]> wrote: >>> >>> > On 17/06/18 16:46, Huxing Zhang wrote: >>> > > Hi Community, >>> > > >>> > > I suggest we review the pull request merge policy. I've seen pull >>> > > request merged but break the build. [1] >>> > > >>> > > My suggestion: >>> > > >>> > > 0. Change must use pull request, even for committers. >>> > > 1. pull request must pass the CI build >>> > > 2. pull request must be merged by another committer >>> > > 3. pull request must pass review from at least one committer >>> > > >>> > > Thoughts? >>> > >>> > My recommendation is don't do it. >>> > >>> > You are creating multiple barriers to progress. Generally, social >>> > controls result in a better community than technical controls. >>> > >>> > My suggestion is to ensure that emails about broken CI builds go to the >>> > dev list and that there is a community expectation that if you break the >>> > build you fix it ASAP. Further, if the build isn't fixed in a timely >>> > manner then it should be acceptable for any committer to revert the >>> > commit that broke the build if they wish. >>> > >>> +1 for this. >>> >>> > >>> > Of course, I'm not a committer here and this is just a recommendation. >>> > >>> > Mark >>> > > > -- > Best Regards！ > Huxing -- Best Regards！ Huxing
