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

All: normalize license notice in all *.rs files #5199

Merged
merged 1 commit into from
Aug 24, 2023

Conversation

tertsdiepraam
Copy link
Member

@tertsdiepraam tertsdiepraam commented Aug 24, 2023

Since we had some trouble finding all authors in #5197, I thought it'd be nice to normalize all notices to exactly the same string. This meant place the comment all at the very top of the file and removing * symbols or additional spaces and other inconsistent formatting. This change should make it easier for us to check that all files have the notice or to change it if we are ever inclined to do so. Also it's just more satisfying that they're all the same :)

You can check that all files are correct with this nu oneliner:

glob src/**/*.rs tests/**/*.rs | filter { |it| not ($it | str ends-with "main.rs") } | each { |it| open --raw $it | lines | range 0..3 | str join "\n" } | uniq

(The main.rs files are omitted but we could add the notice there too)

@sylvestre sylvestre merged commit ffbfea4 into uutils:main Aug 24, 2023
44 of 45 checks passed
@sylvestre
Copy link
Sponsor Contributor

much better, thanks

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