From 35716347196fa041e9ccbe02df3bcff455af4438 Mon Sep 17 00:00:00 2001 From: liangbowen Date: Mon, 7 Aug 2023 18:14:01 +0800 Subject: [PATCH] [KYUUBI #5135] [Minor] Exclude web-ui coverage reports in rat plugins ### _Why are the changes needed?_ - To avoid the generated web-ui's coverage report files triggering Rat plugin checks on Maven ``` [WARNING] Files with unapproved licenses: kyuubi-server/web-ui/coverage/index.html kyuubi-server/web-ui/coverage/locales/en_US/index.html kyuubi-server/web-ui/coverage/locales/zh_CN/index.html ... [ERROR] Failed to execute goal org.apache.rat:apache-rat-plugin:0.15:check (default) on project kyuubi-parent: Too many files with unapproved license: 21 See RAT report in: /kyuubi/target/rat.txt -> ``` ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [x] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request Closes #5135 from bowenliang123/rat-exclude. Closes #5135 1ec4411cb [liangbowen] add web-ui/coverage/* to rat exclusions Authored-by: liangbowen Signed-off-by: liangbowen --- .rat-excludes | 1 + 1 file changed, 1 insertion(+) diff --git a/.rat-excludes b/.rat-excludes index 356fafccd..5735ba69b 100644 --- a/.rat-excludes +++ b/.rat-excludes @@ -53,6 +53,7 @@ build/scala-*/** **/*.sqlite.sql **/node/** **/web-ui/dist/** +**/web-ui/coverage/** **/pnpm-lock.yaml **/node_modules/** **/gen/*