Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cross-project-issues-dev] 1000 line limit for contributions

Yes. This was agreed and approved last year if I recall. 

Legal wanted to check if the 70 or so committers that had limited status would be okey to go unlimited but apparently there are companies that want to limit their contributions :/

I just wish we could make this happen now for those that don't have this limited status.  
Even if it would be a manual lookup so 2 weeks could be reduced to a few days.

Been waiting a year for the automation to come in place. 

Denis, what's the holdup ? :)

On 20 Nov 2015, at 19:36, John Arthorne <John_Arthorne@xxxxxxxxxx> wrote:

The "allow committers to contribute to other eclipse projects" topic is well underway in the Eclipse Board. There is a problem that some committers have written agreements from their employer that only allows them to participate on a particular project. The Foundation needs to put some automation in place to handle this, but there is a path forward here and progress is being made.

John


-----cross-project-issues-dev-bounces@xxxxxxxxxxx wrote: -----
To: Cross project issues <cross-project-issues-dev@xxxxxxxxxxx>
From: Lars Vogel
Sent by: cross-project-issues-dev-bounces@xxxxxxxxxxx
Date: 11/20/2015 11:44AM
Subject: Re: [cross-project-issues-dev] 1000 line limit for contributions

Great proposal from Max to skip CQ for Eclipse committers. It does not make sense that I can commit 100 000 lines to PDE but not 1001 to another project I'm not a committer for.

Max, can you bring your proposal to the board?

Best regards, Lars

Am 20.11.2015 9:00 vorm. schrieb "Max Rydahl Andersen" <manderse@xxxxxxxxxx>:
On 19 Nov 2015, at 11:17, Ed Merks wrote:

Recall that this was increased from 250 to 1000 not so long ago.  I wasn't able to push the IP committee beyond that.  People working for an organization that has a corporate representative on the board could ask that representative to raise this issue at a board meeting.  I think that would carry more weight than me personally asking for a further increase.

In this case it looks like a broken notification - good it was found and identified.

I can personally vouch that the 1000 line limit have directly taken part in why I have stopped contributing to i.e. mylyndoc asciidoc support. Here I spent several weekends and eventually months on getting basic contributions in that was *obviously* not in any danger of having IP conflicts since the code was copied from mylyn.doc itself and 100% written by me - still I had to play the game of splitting up contributions and in the end just stop doing it. (mylyn.doc could fix this by making me a committer or eclipse foundation allow contributions from other eclipse committers without CQ review - but that is just examples of more process dancing)

Thus this really is an issue and something I raised to IP and Legal team several time in the pass - issue is that from their perspective they don't get to see the times a committer is asked to split a contribution up; they just see the +1000 lines ones and think they are fast to get it though the system.

But If I have to wait 2 weeks between contributions for things that are not on my critical path I just cannot afford spending time on it - especially if I have interest in building on top of these contributions.
And I'm actually a believer in doing things right at eclipse - but I can just imagine those coming from the outside just never show up or just leave immediately.

But to the point on raising this to the IP committee and the board.

If you get caught in similar dead or live-locks in the IP system or see another attempt on having to split up otherwise perfect valid contributions, please consider forwarding me info personally (manderse@xxxxxxxxxx).

I would like to be able to show to IP committee and board how often this is actually hurting us - or on the flip side, be convinced that is not actually that big an issue.

Thanks,
/max



On 19/11/2015 11:00 AM, Ed Willink wrote:
Hi

Presumably you put tests in a separate plugin, so splitting off the tests as a separate contribution gets you twice the limit with minimal effort.

Perhaps a 10000 line limit might be appropriate for non-deliverable code such as tests and build tools.

Regards

 Ed Willink



On 19/11/2015 09:49, Sievers, Jan wrote:
Hi,

in the course of

https://bugs.eclipse.org/bugs/show_bug.cgi?id=477328


we had a contribution that slightly exceeded 1000 lines and thus needed a CQ.
It took about one month to review it.

I am sure the legal team does its very best to keep up with the load, so the following is in no way a criticism of the
people who actually do the legal review.

Rather take it as food for thought to whoever set up this rule.

IMHO the 1000 line rule is effectively setting the wrong incentives for a thriving opensource project.

Here is why I think so:


The most diligent contributors add a lot of tests to their patch to prove it works.
This is a good thing and we actively encourage contributors to thoroughly test.
Test code can easily outweigh productive code being tested in terms of LOC.
However this means the most diligent contributors, i.e. the ones you want to attract, are more likely to hit the 1000 line limit.
Instead of thanking them for their hard work, we effectively punish them with an extra month or more wait time before their patch can be merged.
Apart from that, the 1000 line limit seems arbitrary to me because technically you can split up any commit into any number
of smaller commits below the 1000 line limit.

Best Regards,
Jan




_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev


_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev

_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev


/max
http://about.me/maxandersen
_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev

_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev

Back to the top