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

docs: Update license year to 2024 #6451

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

Conversation

joshka
Copy link
Contributor

@joshka joshka commented Apr 2, 2024

This removes the copyright year in all the LICENSE files.
It also uses the MIT license text with the wrapping from
https://choosealicense.com/licenses/mit/ rather than the more narrow
wrapped text that was previously used, and adds the explicit "MIT
License" header to each file to make this more obvious.

Motivation

When I opened the license file, I was unsure if this was MIT licensed or another custom license due to the lack of header and non-standard formatting.

Solution

Use a more standard formatted license, and ensure the header states this is an MIT License.

This updates the license year to 2024 in all the LICENSE files.
It also uses the MIT license text with the wrapping from
https://choosealicense.com/licenses/mit/ rather than the more narrow
wrapped text that was previously used, and adds the explicit "MIT
License" header to each file to make this more obvious.
@Darksonn Darksonn added the A-readme Area: Documentation that isn't part of any crate such as README.md or CONTRIBUTING.md label Apr 2, 2024
@Darksonn
Copy link
Contributor

Darksonn commented Apr 2, 2024

We may want to just drop the year ...

Thoughts @carllerche ?

@joshka
Copy link
Contributor Author

joshka commented Apr 2, 2024

I think that's fine in the US (IANAL however), but perhaps not everywhere?

@Darksonn
Copy link
Contributor

Darksonn commented Apr 2, 2024

The Rust project itself already did the same.

@joshka
Copy link
Contributor Author

joshka commented Apr 3, 2024

The Rust project itself already did the same.

That's good enough rationale for me. Will remove.

@joshka
Copy link
Contributor Author

joshka commented Apr 9, 2024

See also tokio-rs/tracing#2922

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-readme Area: Documentation that isn't part of any crate such as README.md or CONTRIBUTING.md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants