-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[v1.x]Enhance license checker to cover multiple license header and md files #18634
Conversation
Hey @ciyongch , Thanks for submitting the PR
CI supported jobs: [edge, unix-cpu, windows-cpu, miscellaneous, clang, unix-gpu, website, centos-gpu, windows-gpu, sanity, centos-cpu] Note: |
[centos-cpu] [centos-gpu][windows-gpu] jobs failed due to numpy 1.19.0 is used in CI, which is a known issue reported in #18600. @mxnet-bot run ci [unix-gpu] |
Undefined action detected. |
@mxnet-bot run ci [unix-gpu] |
Jenkins CI successfully triggered : [unix-gpu] |
Hi @szha , may I have your suggestion for the failed numpy UTs,
|
+1 for option 1 since the integration indeed needs fix. Thanks for driving this. |
#18651 was created as described in option 1. |
@mxnet-bot run ci [unix-gpu] |
Jenkins CI successfully triggered : [unix-gpu] |
…apache#18634) Co-authored-by: Leonard Lausen <lausen@amazon.com>
Description
Backport #18478 and #18541 to enhance the current tools/icense_header.py, the main changes are listed as below:
@szha @leezu @TaoLv @pengzhao-intel