Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MAINT: change license to 2.0 #235

Merged
merged 1 commit into from
Oct 16, 2018
Merged

MAINT: change license to 2.0 #235

merged 1 commit into from
Oct 16, 2018

Conversation

SantiagoTorres
Copy link
Member

Fixes issue #:
No issue number (discussed on the private mailing list)

Description of the changes being introduced by the pull request:

This pull request updates the license to be Apache 2.0. All
the developers involved in the process have been notified and agreed on
this change

Please verify and check that the pull request fulfills the following
requirements
:

  • [N/A] The code follows the Code Style Guidelines
  • [N/A] Tests have been added for the bug fix or new feature
  • Docs have been added for the bug fix or new feature

@SantiagoTorres
Copy link
Member Author

ping @alyptik @lukpueh @JustinCappos @alyptik

Should be a couple of secs but I don't want to merge it myself...

@lukpueh
Copy link
Member

lukpueh commented Oct 16, 2018

LGTM. GitHub still labels the repo as MIT repo, but the detection routine might only run on the main branch? We'll see once it's merged.

Also Appveyor seems to have fallen victim to our annoying gpg unit test issue #171. I say we ignore the result.

@lukpueh lukpueh merged commit d0111c1 into develop Oct 16, 2018
@lukpueh
Copy link
Member

lukpueh commented Oct 16, 2018

While clicking merge, I realized that the PR misses a few minor changes. Please review and merge in #237.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bbd71f5 on change-license into 9fb1a4a on develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants