devfile / api

Kube-native API for cloud development workspaces specification
Apache License 2.0
237 stars 58 forks source link

Change license to Apache 2.0 #547

Closed l0rd closed 2 years ago

l0rd commented 2 years ago

What does this PR do?

We are about to propose Devfile as a CNCF sandbox project. Most of the CNCF projects are licensed under Apache-2.0 and that’s what the community is used to. Re-licensing to Apache-2.0 would make the process smoother and augment our chances to successfully transition the project to CNCF sandbox. Both EPL-2.0 and Apache-2.0 are open source licenses that are widely used in the industry and Apache-2.0 is considered more permissive than EPL-2.0.

⚠️ This PR is in draft mode. We are contacting existing contributors to get their approval and then we will merge it.

What issues does this PR fix or reference?

https://github.com/devfile/api/issues/426

Contributors list (checked ones have approved the re-licensing)

pierreprinetti commented 2 years ago

LGTM

openshift-ci[bot] commented 2 years ago

@metacosm: changing LGTM is restricted to collaborators

In response to [this](https://github.com/devfile/api/pull/547#pullrequestreview-710532394): > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
GeekArthur commented 2 years ago

I approve.

Jingfu Wang Red Hat Software Engineer Email: @.***

On Tue, Jul 20, 2021 at 7:02 AM Mario Loriedo @.***> wrote:

What does this PR do?

We are about to propose Devfile as a CNCF sandbox project. Most of the CNCF projects are licensed under Apache-2.0 and that’s what the community is used to. Re-licensing to Apache-2.0 would make the process smoother and augment our chances to successfully transition the project to CNCF sandbox. Both EPL-2.0 and Apache-2.0 are open source licenses that are widely used in the industry and Apache-2.0 is considered more permissive than EPL-2.0.

⚠️ This PR is in draft mode. We are contacting existing contributors to get their approval and then we will merge it. What issues does this PR fix or reference?

426 https://github.com/devfile/api/issues/426

Contributors list (checked ones have approved the re-licensing)


You can view, comment on, or merge this pull request online at:

https://github.com/devfile/api/pull/547 Commit Summary

  • Change license to Apache 2.0

File Changes

Patch Links:

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/devfile/api/pull/547, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADTOHEOKX4S4YIVBVMBDPALTYVJTBANCNFSM5AVRRJYA .

mohammedzee1000 commented 2 years ago

I approve

girishramnani commented 2 years ago

I don’t know why I received the email, but I approve

l0rd commented 2 years ago

I don’t know why I received the email, but I approve

@girishramnani maybe for this contribution https://github.com/devfile/library/pull/92?

ranakan19 commented 2 years ago

I believe i had given my approval before, but since i don't see my name in the list of ones who have approved, here it is: I approve.

Regards, Kanika Rana

On Mon, Sep 13, 2021 at 7:00 AM Mario Loriedo @.***> wrote:

I don’t know why I received the email, but I approve

@girishramnani https://github.com/girishramnani maybe for this contribution devfile/library#92 https://github.com/devfile/library/pull/92??

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/devfile/api/pull/547#issuecomment-918077848, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALEZUETLEJ2VUZIH35ZNYNDUBXKUFANCNFSM5AVRRJYA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- Best, Kanika

l0rd commented 2 years ago

I believe i had given my approval before, but since i don't see my name in the list of ones who have approved, here it is: I approve. Regards, Kanika Rana

@ranakan19 I can see you checked in the list above. The only missing approval is @Nadianw36 (and @nwaid but I think that's the same user) but I wasn't able to reach her. Can someone help?

wtam2018 commented 2 years ago

LGTM

l0rd commented 2 years ago

@amisevsk thanks for #630

I got an approval from @Nadianw36, we are ready to merge now. We only need an approval on this PR.

openshift-ci[bot] commented 2 years ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: benoitf, l0rd, metacosm

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/devfile/api/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment