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

Potential license issue #32

Closed
proehlen opened this issue Oct 1, 2023 · 3 comments · Fixed by #33
Closed

Potential license issue #32

proehlen opened this issue Oct 1, 2023 · 3 comments · Fixed by #33

Comments

@proehlen
Copy link

proehlen commented Oct 1, 2023

Hi, I appreciate the work you've put into this project and that you've made it publicly available. However, I notice that 65 or so of the files in your project contain " All rights reserved. MIT license." in the header.

https://github.com/Im-Beast/deno_tui/blob/main/src/components/table.ts

It's my understanding that the use of the phrase "All rights reserved" is, at best, redundant or, at worst, contradictory with the MIT license:

https://opensource.stackexchange.com/a/4403

https://g.co/bard/share/3e4c8688afbf

IANAL and I assumed you copied this usage from another project but I can't unfortunately use your code while the license is seemingly ambiguous. Regardless of whether you decide to change it or not, thanks for considering this issue.

@Im-Beast
Copy link
Owner

Im-Beast commented Oct 1, 2023

I'll gladly change that, I'm pretty sure I just copied that phrase from somewhere, thanks for noticing!

Im-Beast added a commit that referenced this issue Oct 1, 2023
@Im-Beast
Copy link
Owner

Im-Beast commented Oct 1, 2023

It seems like deno_std uses the same phrase, you should probably report an issue about it there too, e.g. https://github.com/denoland/deno_std/blob/5700c3deeba37616d34ea389addf42e163897df9/csv/mod.ts#L1

@proehlen
Copy link
Author

proehlen commented Oct 1, 2023

It seems like deno_std uses the same phrase, you should probably report an issue about it there too, e.g. https://github.com/denoland/deno_std/blob/5700c3deeba37616d34ea389addf42e163897df9/csv/mod.ts#L1

Will do and thanks for taking the time to look at this.

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 a pull request may close this issue.

2 participants