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

Add missing LICENSE files to several workspace crates #1003

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

musicinmybrain
Copy link
Contributor

In the case of crates published on crates.io, like rioterm, rio-proc-macros, teletypewriter, etc., this fixes missing license text in the published crates. Normally, I would just use symbolic links to the top-level LICENSE file in a PR like this, but I followed the lead of existing license files like corcovado/LICENSE and made copies instead.

I also added the LICENSE file to frontends/wasm. It isn’t published on crates.io, and I don’t entirely understand how to use it, but I assumed it was the right thing to do there as well.

Including the license text with published crates is necessary so that people can comply with the chosen MIT license, which says:

rio/LICENSE

Lines 12 to 13 in 73f0158

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

Copy link
Owner

@raphamorim raphamorim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @musicinmybrain

@raphamorim raphamorim merged commit 51a9119 into raphamorim:main Mar 6, 2025
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.

2 participants