This is an automated email from the ASF dual-hosted git repository.
bowenliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/master by this push:
new 357163471 [KYUUBI #5135] [Minor] Exclude web-ui coverage reports in
rat plugins
357163471 is described below
commit 35716347196fa041e9ccbe02df3bcff455af4438
Author: liangbowen <[email protected]>
AuthorDate: Mon Aug 7 18:14:01 2023 +0800
[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 <[email protected]>
Signed-off-by: liangbowen <[email protected]>
---
.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/*
