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

Ensure license headers match the Cargo manifest licenses #5776

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

rzadp
Copy link
Contributor

@rzadp rzadp commented Sep 19, 2024

What's inside

First, we change the file traversal mechanism from shell globbing to walking through files which happens inside the license-scanner - so it has knowledge about directory structure and can correlate files with corresponding Cargo manifest.

Next, added MIT-0 and Unlicense to the allowed list of licenses.
Unlicense appears only once across {Substrate,Cumulus,Polkadot} - in penpal-runtime.

Finally, updated headers in files that do not match the corresponding manifest license.

@rzadp rzadp added the R0-silent Changes should not be mentioned in any release notes label Sep 19, 2024
@rzadp rzadp requested review from a team and koute as code owners September 19, 2024 16:49
@paritytech-review-bot paritytech-review-bot bot requested a review from a team September 19, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compare license to Cargo.toml
4 participants