Rat-check: exclude venv, cleanup excludes, include .svg (apache#2363)

* `.svg` files are XML files and can contain a license header
* Re-grouped the exclusion rat patterns
* Added exclude for `.venv`
* Added exclude for `.ruff_cache`

Author: snazy

build.gradle.kts

@@ -60,11 +60,15 @@ if (System.getProperty("idea.sync.active").toBoolean()) {
 eclipse { project { name = ideName } }
 
 tasks.named<RatTask>("rat").configure {
-  // These are Gradle file pattern syntax
+  // Gradle
   excludes.add("**/build/**")
+  excludes.add("gradle/wrapper/gradle-wrapper*")
+  excludes.add(".gradle")
+  excludes.add("**/kotlin-compiler*")
+  excludes.add("**/build-logic/.kotlin/**")
 
-  excludes.add("docs/CNAME")
-  excludes.add("docs/index.html")
+  excludes.add("ide-name.txt")
+  excludes.add("version.txt")
 
   excludes.add("DISCLAIMER_WIP")
   excludes.add("LICENSE")
@@ -85,60 +89,56 @@ tasks.named<RatTask>("rat").configure {
   // Manifest files do not allow comments
   excludes.add("tools/version/src/jarTest/resources/META-INF/FAKE_MANIFEST.MF")
 
-  excludes.add("ide-name.txt")
-  excludes.add("version.txt")
+  // Git & GitHub
   excludes.add(".git")
-  excludes.add(".gradle")
-  excludes.add(".idea")
-  excludes.add(".java-version")
-  excludes.add("**/.keep")
-  excludes.add("**/poetry.lock")
-
   excludes.add(".github/pull_request_template.md")
 
-  excludes.add("spec/docs.yaml")
-  excludes.add("spec/index.yml")
-
-  excludes.add("gradle/wrapper/gradle-wrapper*")
-
+  // Misc build artifacts
+  excludes.add(".java-version")
+  excludes.add("**/.keep")
   excludes.add("logs/**")
+  excludes.add("**/*.lock")
+
+  // Polaris service startup banner
   excludes.add("runtime/service/src/**/banner.txt")
 
+  // Web site
+  excludes.add("**/go.sum")
   excludes.add("site/node_modules/**")
   excludes.add("site/layouts/robots.txt")
   // Ignore generated stuff, when the Hugo is run w/o Docker
   excludes.add("site/public/**")
   excludes.add("site/resources/_gen/**")
   excludes.add("node_modules/**")
 
+  // Python
+  excludes.add("**/.venv/**")
   excludes.add("**/polaris-venv/**")
-
+  excludes.add("**/poetry.lock")
+  excludes.add("**/.ruff_cache/**")
+  excludes.add("**/.mypy_cache/**")
   excludes.add("**/.pytest_cache/**")
+  excludes.add("client/python/.openapi-generator/**")
+
+  // Jupyter
+  excludes.add("**/*.ipynb")
+
+  // regtests
   excludes.add("regtests/**/py.typed")
   excludes.add("regtests/**/*.ref")
   excludes.add("regtests/.env")
   excludes.add("regtests/derby.log")
   excludes.add("regtests/metastore_db/**")
-  excludes.add("client/python/.openapi-generator/**")
   excludes.add("regtests/output/**")
+  excludes.add("plugins/**/*.ref")
 
-  excludes.add("**/*.ipynb")
+  // IntelliJ
+  excludes.add(".idea")
   excludes.add("**/*.iml")
   excludes.add("**/*.iws")
 
+  // Rat can't scan binary images
   excludes.add("**/*.png")
-  excludes.add("**/*.svg")
-
-  excludes.add("**/*.lock")
-
-  excludes.add("**/*.env*")
-
-  excludes.add("**/go.sum")
-
-  excludes.add("**/kotlin-compiler*")
-  excludes.add("**/build-logic/.kotlin/**")
-
-  excludes.add("plugins/**/*.ref")
 }
 
 tasks.register<Exec>("regeneratePythonClient") {