Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [january-dev] About to create CQ for initial contribution

Hi All,

As you probably saw in my last email I submitted the CQ (first attempt
at one anyway).

The corresponding bugzilla is:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=497523

Jonah

~~~
Jonah Graham
Kichwa Coders Ltd.
www.kichwacoders.com


On 7 July 2016 at 21:02, Jonah Graham <jonah@xxxxxxxxxxxxxxxx> wrote:
> Hi Jay,
>
> I have created the CQ for the initial contribution. As you are our
> mentor, I would appreciate your once over to ensure I don't have any
> big holes.
>
> https://dev.eclipse.org/ipzilla/show_bug.cgi?id=11665
>
> Thanks
> Jonah
>
> ~~~
> Jonah Graham
> Kichwa Coders Ltd.
> www.kichwacoders.com
>
>
> On 7 July 2016 at 16:29, Christopher Brooks <cxh@xxxxxxxxxxxxxxxxx> wrote:
>> BTW - Christopher Brooks' Law is: "A programmer is never more productive
>> than when they are deleting code because many fewer bugs are caused by
>> missing code than by incorrect code."
>>
>> This is in support of Brooks' Law: "adding manpower to a late software
>> project makes it later"
>>
>> Instead of adding manpower, remove code.
>>
>> :-)
>>
>> (Fred Brooks is no relation to my family)
>>
>> _Christopher
>>
>>
>> On 7/7/16 7:24 AM, Jay Jay Billings wrote:
>>
>> The PMC said the same. I just accepted the pull request. Thanks Jonah!
>>
>> Jay
>>
>> On Thu, Jul 7, 2016 at 8:48 AM, Jonah Graham <jonah@xxxxxxxxxxxxxxxx> wrote:
>>>
>>> As a reference point, when I removed CDI from CDT (84,000 lines,
>>> https://git.eclipse.org/r/#/c/63282/) no CQ or anything was needed.
>>> When using gerrit, the automatic stuff checks if over 1000 lines were
>>> added. Deleting code is not adding any IP.
>>>
>>> Jonah
>>> ~~~
>>> Jonah Graham
>>> Kichwa Coders Ltd.
>>> www.kichwacoders.com
>>>
>>>
>>> On 7 July 2016 at 13:33, Jay Jay Billings <jayjaybillings@xxxxxxxxx>
>>> wrote:
>>> > I have to check with the PMC about your pull request because it is over
>>> > 1000
>>> > lines. I'll CC the January dev list.
>>> >
>>> > Jay
>>> >
>>> > On Thu, Jul 7, 2016 at 4:42 AM, Jonah Graham <jonah@xxxxxxxxxxxxxxxx>
>>> > wrote:
>>> >>
>>> >> Great.
>>> >>
>>> >> I'll wait until this afternoon to give Jay a chance to merge my PR and
>>> >> then we are good to go.
>>> >>
>>> >> Jonah
>>> >>
>>> >> ~~~
>>> >> Jonah Graham
>>> >> Kichwa Coders Ltd.
>>> >> www.kichwacoders.com
>>> >>
>>> >>
>>> >> On 7 July 2016 at 09:11,  <Peter.Chang@xxxxxxxxxxxxx> wrote:
>>> >> > Hi Jonah,
>>> >> >
>>> >> > That's a fine plan. I may branch off that to add more changes but we
>>> >> > can
>>> >> > merge those later.
>>> >> >
>>> >> > Regards,
>>> >> >  Peter
>>> >> >
>>> >> >
>>> >> > -----Original Message-----
>>> >> > From: january-dev-bounces@xxxxxxxxxxx
>>> >> > [mailto:january-dev-bounces@xxxxxxxxxxx] On Behalf Of Jonah Graham
>>> >> > Sent: 06 July 2016 22:30
>>> >> > To: january developer discussions <january-dev@xxxxxxxxxxx>
>>> >> > Subject: [january-dev] About to create CQ for initial contribution
>>> >> >
>>> >> > Hi January devs,
>>> >> >
>>> >> > I think we now have the three parts of our initial contribution ready
>>> >> > to
>>> >> > go. I think we are ready to start the CQ process.
>>> >> >
>>> >> > The initial contribution will be the org.eclipse.january.* bundles
>>> >> > from
>>> >> > the ICE, Dawnsci and EAVP repositories. These are the git commits I
>>> >> > will be
>>> >> > using.
>>> >> >
>>> >> >
>>> >> >
>>> >> > https://github.com/eclipse/dawnsci/commit/d46a644e0443b4c1ffe37760af86283210372812
>>> >> >
>>> >> >
>>> >> > https://github.com/eclipse/ice/commit/66b4c63f8cf0812c13d30af45ec3851a3c4b9be3
>>> >> >
>>> >> >
>>> >> > https://github.com/eclipse/eavp/commit/8c60bc100ddc12ab5441a381ea0886fd66ee0361
>>> >> >
>>> >> > The only caveats are that for EAVP I need to remove some files.
>>> >> > - There are some presumably accidentally committed files, I created
>>> >> > https://github.com/eclipse/eavp/pull/74 for that
>>> >> > - The antlr patch needs to be removed from the CQ zip:
>>> >> >
>>> >> >
>>> >> > https://github.com/eclipse/eavp/blob/8c60bc100ddc12ab5441a381ea0886fd66ee0361/org.eclipse.january.geometry.model/.antlr-generator-3.2.0-patch.jar
>>> >> >
>>> >> > If these can be addressed, then I will be able to have a cleaner
>>> >> > correspondence between code in the CQ and git commits.
>>> >> >
>>> >> > For sanity and traceability, I think it is important that these git
>>> >> > commit ids are long lived, or at least long enough until the initial
>>> >> > contribution is approved and the code is moved across. By doing it
>>> >> > that way,
>>> >> > applying any changes made between the initial contribution and the
>>> >> > approval
>>> >> > will be a lot easier.
>>> >> >
>>> >> > How does the above sound as a plan? I will raise the CQ tomorrow
>>> >> >
>>> >> > Thank you everyone for your hard work on getting January ready! I
>>> >> > think
>>> >> > we are nearly there.
>>> >> >
>>> >> > Jonah
>>> >> >
>>> >> > ~~~
>>> >> > Jonah Graham
>>> >> > Kichwa Coders Ltd.
>>> >> > www.kichwacoders.com
>>> >> > _______________________________________________
>>> >> > january-dev mailing list
>>> >> > january-dev@xxxxxxxxxxx
>>> >> > To change your delivery options, retrieve your password, or
>>> >> > unsubscribe
>>> >> > from this list, visit
>>> >> > https://dev.eclipse.org/mailman/listinfo/january-dev
>>> >> >
>>> >> > --
>>> >> > This e-mail and any attachments may contain confidential, copyright
>>> >> > and
>>> >> > or privileged material, and are for the use of the intended addressee
>>> >> > only.
>>> >> > If you are not the intended addressee or an authorised recipient of
>>> >> > the
>>> >> > addressee please notify us of receipt by returning the e-mail and do
>>> >> > not
>>> >> > use, copy, retain, distribute or disclose the information in or
>>> >> > attached to
>>> >> > the e-mail.
>>> >> > Any opinions expressed within this e-mail are those of the individual
>>> >> > and not necessarily of Diamond Light Source Ltd.
>>> >> > Diamond Light Source Ltd. cannot guarantee that this e-mail or any
>>> >> > attachments are free from viruses and we cannot accept liability for
>>> >> > any
>>> >> > damage which you may sustain as a result of software viruses which
>>> >> > may be
>>> >> > transmitted in or with the message.
>>> >> > Diamond Light Source Limited (company no. 4375679). Registered in
>>> >> > England and Wales with its registered office at Diamond House,
>>> >> > Harwell
>>> >> > Science and Innovation Campus, Didcot, Oxfordshire, OX11 0DE, United
>>> >> > Kingdom
>>> >> >
>>> >> > _______________________________________________
>>> >> > january-dev mailing list
>>> >> > january-dev@xxxxxxxxxxx
>>> >> > To change your delivery options, retrieve your password, or
>>> >> > unsubscribe
>>> >> > from this list, visit
>>> >> > https://dev.eclipse.org/mailman/listinfo/january-dev
>>> >> _______________________________________________
>>> >> january-dev mailing list
>>> >> january-dev@xxxxxxxxxxx
>>> >> To change your delivery options, retrieve your password, or unsubscribe
>>> >> from this list, visit
>>> >> https://dev.eclipse.org/mailman/listinfo/january-dev
>>> >
>>> >
>>> >
>>> >
>>> > --
>>> > Jay Jay Billings
>>> > Oak Ridge National Laboratory
>>> > Twitter Handle: @jayjaybillings
>>> >
>>> > _______________________________________________
>>> > january-dev mailing list
>>> > january-dev@xxxxxxxxxxx
>>> > To change your delivery options, retrieve your password, or unsubscribe
>>> > from
>>> > this list, visit
>>> > https://dev.eclipse.org/mailman/listinfo/january-dev
>>> >
>>> _______________________________________________
>>> january-dev mailing list
>>> january-dev@xxxxxxxxxxx
>>> To change your delivery options, retrieve your password, or unsubscribe
>>> from this list, visit
>>> https://dev.eclipse.org/mailman/listinfo/january-dev
>>
>>
>>
>>
>> --
>> Jay Jay Billings
>> Oak Ridge National Laboratory
>> Twitter Handle: @jayjaybillings
>>
>>
>> _______________________________________________
>> january-dev mailing list
>> january-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or unsubscribe from
>> this list, visit
>> https://dev.eclipse.org/mailman/listinfo/january-dev
>>
>>
>> --
>> Christopher Brooks, PMP                       University of California
>> Academic Program Manager & Software Engineer  US Mail: 337 Cory Hall
>> CHESS/iCyPhy/Ptolemy/TerraSwarm               Berkeley, CA 94720-1774
>> cxh@xxxxxxxxxxxxxxxxx, 707.332.0670           (Office: 545Q Cory)
>>
>>
>> _______________________________________________
>> january-dev mailing list
>> january-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or unsubscribe from
>> this list, visit
>> https://dev.eclipse.org/mailman/listinfo/january-dev
>>


Back to the top