Skip to content

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

Merged
sylvestre merged 1 commit intouutils:mainfrom
tertsdiepraam:normalize-license-notice
Aug 24, 2023
Merged

All: normalize license notice in all *.rs files#5199
sylvestre merged 1 commit intouutils:mainfrom
tertsdiepraam:normalize-license-notice

Conversation

@tertsdiepraam
Copy link
Copy Markdown
Collaborator

@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
@sylvestre
Copy link
Copy Markdown
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