-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add license check #3268
Add license check #3268
Conversation
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.
LGTM
Run in CI? |
Request CI |
Ignore the license verification of cmake, conf, tests, and docker for now, and you can open it if necessary |
Codecov Report
@@ Coverage Diff @@
## master #3268 +/- ##
==========================================
- Coverage 85.25% 85.24% -0.01%
==========================================
Files 1295 1295
Lines 118193 118201 +8
==========================================
+ Hits 100760 100763 +3
- Misses 17433 17438 +5
Continue to review full report at Codecov.
|
Need fix build error for self-hosted |
What type of PR is this?
What does this PR do?
Add license check
Which issue(s)/PR(s) this PR relates to?
None