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 LICENSE file to nom-language #1816

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

miguelfrde
Copy link

@miguelfrde miguelfrde commented Feb 5, 2025

Hi, I noticed that the nom-language package doesn't contain a LICENSE file. Including it would make it easier to use in other codebases.

This change creates a symlink to the root LICENSE file.

This ensures that the LICENSE text is contained in the cargo package output. With this change after cargo package the nom-language crate:

$ cd target/package/nom-language-0.1.0
$ ls
Cargo.lock      Cargo.toml.orig src
Cargo.toml      LICENSE         target

This matches what multiple other repositories do (for example, actix-web, tracing-core).

This is just a copy of the root LICENSE file
@miguelfrde miguelfrde requested a review from Geal as a code owner February 5, 2025 22:31
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.

1 participant