[tractusx-dev] Contributions above 1.000 lines of code

["Kiermayer, Siegfried" <siegfried.kiermayer@xxxxxxx>]

Hello community,

 

I would like to make all of us aware that we do have a high responsibility for contributions which come from outside.

 

Outside means from people who are not committers which can include friends, family, and co-worker.

 

The Eclipse Handbook is very clear about contributions from outside with changes more than 1.000 lines of code:  https://www.eclipse.org/projects/handbook/#ip-project-content

 

“Further investigation is required when any of the following conditions are met:

• The content has been copied, or is based on content that has been copied (that is, the content was not 100% authored by the submitting contributor);
• The licence terms of the content are other than the terms of the project license;
• The content contains cryptography; or
• The content contains more than 1,000 lines of content (including documentation, code, configuration files, and other forms of source code).

When further investigation is required, a project committer must engage with the IP Team to request their review before the contribution is pushed/merged.”

 

Even changes to documentation and configuration files are already included in the eclipse handbook. Do not assume that ‘just because it was this type of change or that type of change’ that you don’t need to create a ticket with the IP Team.

 

The benefit of committers and committers only:

“Contributions made by Eclipse committers do not require review by the Eclipse IP Team and can be pushed directly into a project repository. Notwithstanding project-specific rules, project committers can push their own content directly into project repositories. While content produced by other Eclipse committers does not require review by the Eclipse IP Team, it must still be accepted by project committers”

 

The most critical thing, we as indirect representatives of the Eclipse Foundation, is to make sure that we do our due diligence regarding IP Checks. Do not think about this as optional.

 

Thanks,

 

Sigi