Address comments

apache · Dec 4, 2024 · 937d3b4 · 937d3b4
1 parent 973016d
commit 937d3b4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 22 deletions.
diff --git a/.github/workflows/python-ci-docs.yml b/.github/workflows/python-ci-docs.yml
@@ -23,20 +23,7 @@ on:
   push:
     branches:
       - 'main'
-  pull_request:
-    paths:
-    - '**'
-    - '!.github/ISSUE_TEMPLATE/**'
-    - '!.github/workflows/*'
-    - '.github/workflows/python-ci-docs.yml'
-    - '!.gitignore'
-    - '!.asf.yml'
-    - '!mkdocs/**'
-    - '!.gitattributes'
-    - '!README.md'
-    - '!CONTRIBUTING.md'
-    - '!LICENSE'
-    - '!NOTICE'
+  pull_request:    
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}

diff --git a/.github/workflows/python-ci.yml b/.github/workflows/python-ci.yml
@@ -25,10 +25,10 @@ on:
     - 'main'
   pull_request:
     paths:
-    - '**'
-    - '!.github/ISSUE_TEMPLATE/**'
-    - '!.github/workflows/**'
-    - '.github/workflows/python-ci.yml'
+    - '**'                              # Include all files and directories in the repository by default. 
+    - '!.github/workflows/**'           # Exclude all workflow files 
+    - '.github/workflows/python-ci.yml' # except the current file. 
+    - '!.github/ISSUE_TEMPLATE/**'      # Exclude files and directories that don't impact tests or code like templates, metadata, and documentation.
     - '!.gitignore'
     - '!.asf.yml'
     - '!mkdocs/**'

diff --git a/.github/workflows/python-integration.yml b/.github/workflows/python-integration.yml
@@ -25,10 +25,10 @@ on:
     - 'main'
   pull_request:
     paths:
-    - '**'
-    - '!.github/ISSUE_TEMPLATE/**'
-    - '!.github/workflows/**'
-    - '.github/workflows/python-integration.yml'
+    - '**'                                       # Include all files and directories in the repository by default. 
+    - '!.github/workflows/**'                    # Exclude all workflow files 
+    - '.github/workflows/python-integration.yml' # except the current file. 
+    - '!.github/ISSUE_TEMPLATE/**'               # Exclude files and directories that don't impact tests or code like templates, metadata, and documentation.
     - '!.gitignore'
     - '!.asf.yml'
     - '!mkdocs/**'