-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[LICENSE] updated licenses for files under both ASF + BSD 3-clause #20495
Conversation
Hey @hsl89 , Thanks for submitting the PR
CI supported jobs: [windows-cpu, clang, website, centos-cpu, unix-gpu, sanity, miscellaneous, windows-gpu, unix-cpu, edge, centos-gpu] Note: |
@hsl89 , A side note: If you put x instead of Y in the Essentials and Changes' list (like [x]), it will show a check mark there. |
magic |
Also noticed that FindJeMalloc.cmake is failing the license check as well, since |
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.
Thanks! LGTM!
@hsl89 Can you please port this to v1.x and master branches? Thanks! |
Description
This PR is for item 4 in #20475
Checklist
Essentials
Changes