Great! I plan to start submitting my commits for review - I’ll figure out a process that works for me (rbt, web posting, etc) and post my thoughts. One thing that will probably be important is managing the scope of commits - reviews should be kept to a manageable size (ideally around 5 files or so). That means that some JIRAs will probably be best addressed by multiple incremental commits rather than one large one (if portions can be broken into commits that do not break the overall functionality).
— Jon On Jan 30, 2015, at 9:18 AM, Ted Yu <[email protected]> wrote: > The INFRA JIRA has been resolved. > > We should be able to use reviewboard now. > > > >> On Jan 27, 2015, at 10:34 AM, Steve Loughran <[email protected]> wrote: >> >> infra all hang out on hipchat >> >> http://cp.mcafee.com/d/5fHCN0gdEI9FK9zDS7PtPqdT7HLIcII6QrK6zBxNB55cSztNWXX3bbVJ6XxEVouu7ffCN5noRw2ytoPH0Nm9mDbCOtoPH0Nm9mDbCPhOwCqei5C7-LPbzPVEVhWZOWrbOvefzD4XYJteOaaJQmul3PWApmU6CQjqpK_9TLuZXTLsTsSjDdqymokUD9qNfpYLx50gA6S1xoqdEupbN-rso5T64kiUyHIqMe9RzeI35oBqsK2G7Og-fixpIxIWNXlJIj_w09JVZV4S2_id44vcOYMq825tywrhsdFzYp2BUD >> >>> On 27 January 2015 at 16:34, Jon Maron <[email protected]> wrote: >>> >>> Ted made that change early in January, but the issue has yet even get >>> assigned. Is there any mechanism available to get this issue some >>> attention? Can we escalate it? >>> >>> — Jon >>> >>> On Jan 5, 2015, at 10:14 AM, Billie Rinaldi <[email protected]> >>> wrote: >>> >>>> It might help (a little) if the component were listed as ReviewBoard. I >>>> think only Ted can change this. >>>> >>>>> On Mon, Jan 5, 2015 at 7:06 AM, Jon Maron <[email protected]> >>>> wrote: >>>> >>>>> Is there a way to poke one of these to get it moving? :) >>>>> >>>>> — Jon >>>>> >>>>>> On Dec 21, 2014, at 11:41 PM, Ted Yu <[email protected]> wrote: >>>>>> >>>>>> Logged INFRA-8927 >>>>>> >>>>>> Cheers >>>>>> >>>>>> On Sun, Dec 21, 2014 at 4:37 PM, Sumit Mohanty < >>> [email protected]> >>>>>> wrote: >>>>>> >>>>>>> You will need an INFRA jira to create a slider review group and also >>>>> link >>>>>>> the git repo for slider. I had created one INFRA jira for Ambari a >>> while >>>>>>> back for the same. See if that can be a reference. >>>>>>> >>>>>>> On Monday, December 22, 2014, Jon Maron <[email protected]> >>> wrote: >>>>>>> >>>>>>>> I’ve wondered about that myself. I’m wondering if anything is really >>>>>>>> required administratively, or can we simply install RBTools and start >>>>>>>> posting reviews? >>>>>>>> >>>>>>>> On Dec 21, 2014, at 4:28 PM, Ted Yu <[email protected] >>>>> <javascript:;>> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Hi, >>>>>>>>> Who should I contact so that patch for Slider can be added to >>>>>>>>> http://cp.mcafee.com/d/avndzgAd6Qm4QT4NPX3VKVJ6XzRTS6mm3qdT3hOMUOyyCrhKUZtZxBBYSztMQsIff3DDPoyHIqM1heIpRwoH4HjBPpeIpRwoH4HjBPpEVgjd792P3_nVBNVYQsEZuVtdBVfD7NPyt-mKDp55mWbfaxVZicHs3jqpJcTvAXTLuZXTKrKr9PCJhbczVCnCatGSS9_M078V3P66QfcBU_dKc2Xz2a9shlSdo74WNDm1yIiJen1l3V8v7FgISgStoZGSS9_M04SY-Yyr1vF6y2fCpuod412KNgdEK6OBw7iKgXPf44 >>>>>>>>> ? >>>>>>>>> >>>>>>>>> Cheers >>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> CONFIDENTIALITY NOTICE >>>>>>>> NOTICE: This message is intended for the use of the individual or >>>>> entity >>>>>>> to >>>>>>>> which it is addressed and may contain information that is >>> confidential, >>>>>>>> privileged and exempt from disclosure under applicable law. If the >>>>> reader >>>>>>>> of this message is not the intended recipient, you are hereby >>> notified >>>>>>> that >>>>>>>> any printing, copying, dissemination, distribution, disclosure or >>>>>>>> forwarding of this communication is strictly prohibited. If you have >>>>>>>> received this communication in error, please contact the sender >>>>>>> immediately >>>>>>>> and delete it from your system. Thank You. >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> thanks >>>>>>> Sumit >>>>> >>>>> >>>>> -- >>>>> CONFIDENTIALITY NOTICE >>>>> NOTICE: This message is intended for the use of the individual or >>> entity to >>>>> which it is addressed and may contain information that is confidential, >>>>> privileged and exempt from disclosure under applicable law. If the >>> reader >>>>> of this message is not the intended recipient, you are hereby notified >>> that >>>>> any printing, copying, dissemination, distribution, disclosure or >>>>> forwarding of this communication is strictly prohibited. If you have >>>>> received this communication in error, please contact the sender >>> immediately >>>>> and delete it from your system. Thank You. >>> >>> >>> -- >>> CONFIDENTIALITY NOTICE >>> NOTICE: This message is intended for the use of the individual or entity to >>> which it is addressed and may contain information that is confidential, >>> privileged and exempt from disclosure under applicable law. If the reader >>> of this message is not the intended recipient, you are hereby notified that >>> any printing, copying, dissemination, distribution, disclosure or >>> forwarding of this communication is strictly prohibited. If you have >>> received this communication in error, please contact the sender immediately >>> and delete it from your system. Thank You. >> >> -- >> CONFIDENTIALITY NOTICE >> NOTICE: This message is intended for the use of the individual or entity to >> which it is addressed and may contain information that is confidential, >> privileged and exempt from disclosure under applicable law. If the reader >> of this message is not the intended recipient, you are hereby notified that >> any printing, copying, dissemination, distribution, disclosure or >> forwarding of this communication is strictly prohibited. If you have >> received this communication in error, please contact the sender immediately >> and delete it from your system. Thank You.
