From 835de147d9437479dd385fc1de0844bd434b278a Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Mon, 18 Nov 2024 14:54:21 +0800 Subject: [PATCH 01/10] add header check workflow Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 44 ++++++++++++++++++++++ 1 file changed, 44 insertions(+) create mode 100644 .github/workflows/license-header-check.yml diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml new file mode 100644 index 0000000000..621bd4f02b --- /dev/null +++ b/.github/workflows/license-header-check.yml @@ -0,0 +1,44 @@ +# Copyright (c) 2024, NVIDIA CORPORATION. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +# A workflow to check copyright/license header +name: license header check + +on: + pull_request: + types: [opened, synchronize, reopened] + +jobs: + license-header-check: + runs-on: ubuntu-latest + steps: + - name: Checkout code + uses: actions/checkout@v4 + with: + fetch-depth: 5 + + - name: license-header-check + uses: YanxuanLiu/spark-rapids-common/license-header-check@license-header-check + with: + included_file_patterns: | + *.cpp, + *.java, + *.scala, + *.py, + *.sh, + */Dockerfile*, + */Jenkinsfile*, + *.yml + excluded_file_patterns: | + */target/* \ No newline at end of file From 192822e1d6230ab641d50de2f67fcd3c8a1aa7c4 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Mon, 18 Nov 2024 16:59:45 +0800 Subject: [PATCH 02/10] ignore patches Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index 621bd4f02b..cfb592675d 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -41,4 +41,5 @@ jobs: */Jenkinsfile*, *.yml excluded_file_patterns: | - */target/* \ No newline at end of file + */target/* + */patches/* \ No newline at end of file From 8ce459347fc3eab43aa84324c8aca562444a7655 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Mon, 18 Nov 2024 17:07:58 +0800 Subject: [PATCH 03/10] exclude Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index cfb592675d..e638f71f05 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -41,5 +41,5 @@ jobs: */Jenkinsfile*, *.yml excluded_file_patterns: | - */target/* - */patches/* \ No newline at end of file + *target/* + *patches/* \ No newline at end of file From 764302c311cc7b0be793cb6de8d97b8b48cbfb61 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Mon, 18 Nov 2024 18:06:45 +0800 Subject: [PATCH 04/10] change exclude Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index e638f71f05..c058ecfcf7 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -37,9 +37,8 @@ jobs: *.scala, *.py, *.sh, - */Dockerfile*, - */Jenkinsfile*, + *Dockerfile*, + *Jenkinsfile*, *.yml excluded_file_patterns: | - *target/* - *patches/* \ No newline at end of file + patches/* \ No newline at end of file From 5c6667c8c27cdc28cac2802cdaf52fbbaa79aed4 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Tue, 19 Nov 2024 13:12:54 +0800 Subject: [PATCH 05/10] update license header check Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index c058ecfcf7..950438da2c 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -33,12 +33,13 @@ jobs: with: included_file_patterns: | *.cpp, + *.hpp, + *.cu, + *.cuh, *.java, - *.scala, - *.py, *.sh, *Dockerfile*, *Jenkinsfile*, - *.yml - excluded_file_patterns: | - patches/* \ No newline at end of file + *.yml, + *.yaml, + *.txt From c906520b7556979c4cdc9a06bfc3d761c0bfe181 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Tue, 19 Nov 2024 13:33:28 +0800 Subject: [PATCH 06/10] change acion branch Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index 950438da2c..fed240ee70 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -29,7 +29,7 @@ jobs: fetch-depth: 5 - name: license-header-check - uses: YanxuanLiu/spark-rapids-common/license-header-check@license-header-check + uses: YanxuanLiu/spark-rapids-common/license-header-check@license-check with: included_file_patterns: | *.cpp, From 62817e8343c77266d46c7ce606bd25aa86db2f2a Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Tue, 19 Nov 2024 13:54:16 +0800 Subject: [PATCH 07/10] change branch Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index fed240ee70..e7ae58ad74 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -29,7 +29,7 @@ jobs: fetch-depth: 5 - name: license-header-check - uses: YanxuanLiu/spark-rapids-common/license-header-check@license-check + uses: NVIDIA/spark-rapids-common/license-header-check@main with: included_file_patterns: | *.cpp, From 834b90b8494bbb3bad88f76eb89ceb635b1ac05d Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Tue, 19 Nov 2024 15:05:59 +0800 Subject: [PATCH 08/10] change branch to test Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index e7ae58ad74..fed240ee70 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -29,7 +29,7 @@ jobs: fetch-depth: 5 - name: license-header-check - uses: NVIDIA/spark-rapids-common/license-header-check@main + uses: YanxuanLiu/spark-rapids-common/license-header-check@license-check with: included_file_patterns: | *.cpp, From 5bafc2af48051bf3a23403d4d117b83865dfb441 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Tue, 19 Nov 2024 16:33:43 +0800 Subject: [PATCH 09/10] change back branch Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index fed240ee70..e7ae58ad74 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -29,7 +29,7 @@ jobs: fetch-depth: 5 - name: license-header-check - uses: YanxuanLiu/spark-rapids-common/license-header-check@license-check + uses: NVIDIA/spark-rapids-common/license-header-check@main with: included_file_patterns: | *.cpp, From 2a74eae345ebd032e63d3f23983d5772ebd91c11 Mon Sep 17 00:00:00 2001 From: YanxuanLiu Date: Wed, 20 Nov 2024 09:36:00 +0800 Subject: [PATCH 10/10] add more included patterns Signed-off-by: YanxuanLiu --- .github/workflows/license-header-check.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/license-header-check.yml b/.github/workflows/license-header-check.yml index e7ae58ad74..a2faa44f83 100644 --- a/.github/workflows/license-header-check.yml +++ b/.github/workflows/license-header-check.yml @@ -42,4 +42,7 @@ jobs: *Jenkinsfile*, *.yml, *.yaml, - *.txt + *.txt, + *.xml, + *.fbs, + build/*