-
Notifications
You must be signed in to change notification settings - Fork 1k
chore: update license headers to use shorter SPDX header #18148
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
Conversation
Signed-off-by: Mike Fiedler <[email protected]>
Signed-off-by: Mike Fiedler <[email protected]>
Signed-off-by: Mike Fiedler <[email protected]>
Signed-off-by: Mike Fiedler <[email protected]>
Signed-off-by: Mike Fiedler <[email protected]>
Signed-off-by: Mike Fiedler <[email protected]>
Signed-off-by: Mike Fiedler <[email protected]>
I'm considering that this might be a good idea to split out part of this PR to separate PRs so that we can use the If you do think it's a good idea, I'd split out to two PRs:
Once merged & deployed, a third PR to add a I don't feel strongly about this, but wanted to surface the ability in case that was a concern. |
CodeQL failures are due to touching files, and thus scanning "changed" files, so nothing is actually new here. |
I'm:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I say go for it, but with git blame ignore handling.
Excellent! Converting back to draft to split apart and ship parts to accomplish the git blame ignore |
Closing now that the deed is done, and the ignore is part of #18219 |
Instead of the longer license header, replace with a shorter SPDX version.
Refs: https://spdx.dev/learn/handling-license-info/