From 6a9789dfd2d983f8edd4da500bef069c96875f0d Mon Sep 17 00:00:00 2001 From: John Mertic Date: Thu, 30 Apr 2026 13:28:48 -0400 Subject: [PATCH] Revise CLA process and details in CONTRIBUTING.md Updated Contributor License Agreement (CLA) section to include details about signing the CLA through EasyCLA and clarified the process for individual and corporate contributors. Signed-off-by: John Mertic --- CONTRIBUTING.md | 46 +++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 41 insertions(+), 5 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 90e6461..58023a3 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -18,10 +18,46 @@ MoonRay is licensed under the [Apache 2.0 license](LICENSE). Contributions to th ### Contributor License Agreement (CLA) -To contribute to MoonRay, you must sign a Contributor License Agreement (CLA). - -* If you are an individual writing the code on your own time and you're sure you are the sole owner of any intellectual property you contribute, you can sign the [ICLA as an Individual Contributor](https://github.com/dreamworksanimation/openmoonray/blob/main/tsc/icla.md). -* If you are writing the code as part of your job, or if your employer retains ownership to intellectual property you create, then your company's legal affairs representatives should sign a [Corporate Contributor License Agreement](https://github.com/dreamworksanimation/openmoonray/blob/main/tsc/ccla.md). If your company already has a signed CCLA on file, ask your local CLA manager to add you to your company's approved contributor list. +Developers who wish to contribute code to be considered for inclusion +in the MoonRay project must first complete a **Contributor +License Agreement**. + +To contribute to MoonRay, you must sign a CLA through the +[EasyCLA](https://docs.linuxfoundation.org/lfx/easycla) +system, which is integrated with GitHub as a pull request check. + +If a contributor opens a pull request without having a CLA on file, the +contributor will be guided through the process to have the appropriate +CLA signed. Look in the PR comments for the "linux-foundation-easycla" +check that would fail, and a red "NOT COVERED" button will appear in the PR +comments; click the link in the comment to sign the CLA. For organizations, +you can alternatively go to [this +link](https://organization.lfx.linuxfoundation.org/foundation/a09410000182dD2AAI/project/a092M00001If9ujQAB/cla) +prior to submitting a pull request, which will guide you through the +process to have a CLA signed on behalf of the organization. + +* If you are an individual writing the code on your own time and + you're **sure** you are the sole owner of any intellectual property you + contribute, you can sign the CLA as an **Individual Contributor**. If you + are unsure, please contact your employer for clarity. + +* If you are writing the code as part of your job, or if your employer + retains ownership to intellectual property you create, no matter how + small, then your company's legal affairs representatives should sign + a **Corporate Contributor Licence Agreement**. If your company already + has a signed CCLA on file, ask your local CLA manager to add you + (via your GitHub account name/email address) to your company's + "approved" list. + +The downloadable PDF's on the EasyCLA page are provided for reference +only. To execute the signature, sign the form online through the +relevant links. + +The MoonRay CLAs are the standard forms used by the Linux Foundation +projects and [recommended by the ASWF +TAC](https://github.com/AcademySoftwareFoundation/tac/blob/main/process/contributing.md#contributor-license-agreement-cla). +Note that if you have signed a CLA for a different ASWF or LF project, that +CLA doesn't apply here and you need to sign for this project specifically. ### Developer Certificate of Origin (DCO) @@ -81,4 +117,4 @@ Most MoonRay modules have a companion `tests` folder within the repository, cont Additionally, MoonRay provides a [Render Acceptance Test Suite (RATS)](https://github.com/dreamworksanimation/rats) to catch visual regressions caused by changes to the codebase before those changes are deployed into a production environment. It works by comparing canonical images rendered with the previously sanctioned version of the renderer to images rendered with a development version. -All contributions that may impact the resulting look of rendered images must pass RATS before being accepted. Unintentional look changes must be addressed in the contributed code. Intentional look changes must be approved by the TSC, and updated canonical reference images must be provided such that the tests continue to pass. \ No newline at end of file +All contributions that may impact the resulting look of rendered images must pass RATS before being accepted. Unintentional look changes must be addressed in the contributed code. Intentional look changes must be approved by the TSC, and updated canonical reference images must be provided such that the tests continue to pass.