Skip to content

Remove license headers from source files#25

Merged
newpavlov merged 2 commits intomasterfrom
rm_headers
Nov 10, 2025
Merged

Remove license headers from source files#25
newpavlov merged 2 commits intomasterfrom
rm_headers

Conversation

@newpavlov
Copy link
Member

The LICENSE files and the license fields are sufficient for license attribution, so license header in source files are arguably redundant for projects without mixed-license code bases.

@newpavlov newpavlov requested a review from dhardy November 10, 2025 17:13
@newpavlov
Copy link
Member Author

BTW I think rand_core does not contain any code from the Rust project anymore, so we probably could remove its attribution.

Copy link
Member

@dhardy dhardy left a comment

Choose a reason for hiding this comment

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

I agree on both counts.

@newpavlov newpavlov merged commit 4d22ab2 into master Nov 10, 2025
13 checks passed
@newpavlov newpavlov deleted the rm_headers branch November 10, 2025 17:37
takumi-earth pushed a commit to earthlings-dev/rand_core that referenced this pull request Jan 27, 2026
The LICENSE files and the `license` field are sufficient for license attribution.

Additionally, removes attribution of the Rust project since the crate does not
contain any code from it anymore.
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