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.

Sign up for GitHub

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

Jump to bottom

License? #13

Closed
saioia-msu opened this issue Jan 30, 2024 · 4 comments · Fixed by #23
Closed

License? #13

saioia-msu opened this issue Jan 30, 2024 · 4 comments · Fixed by #23

Comments

@saioia-msu
Copy link

What license are these scripts released under?
@maurice-cote
Copy link
Contributor

ah right, I will ensure we upload the license.txt ASAP

@maurice-cote maurice-cote linked a pull request Apr 5, 2024 that will close this issue
Missing licenses #23
Merged
@maurice-cote
Copy link
Contributor

Added both APACHE and MIT license files to repo

@saioia-msu
Copy link
Author

@maurice-cote Looks like you forgot to update the copyright notice in the Apache license, and should probably mention in the README that the repository is dual-licensed under both MIT and Apache to avoid confusion.

Thank you for taking a look at this.

@maurice-cote
Copy link
Contributor

all done! I hope this meets community best practices.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Missing licenses Devolutions/PAM-Providers
2 participants
@maurice-cote @saioia-msu