From 23de056b390f9caf2646a41fd4baacf96dd53459 Mon Sep 17 00:00:00 2001
From: Juarez Rudsatz <juarezr@gmail.com>
Date: Thu, 14 Mar 2024 20:04:14 -0300
Subject: [PATCH] ci: use pull_request as trigger to code analysis workflows

---
 .github/workflows/codacy-analysis.yml | 10 ++++----
 .github/workflows/codeql-analysis.yml | 10 ++++----
 .github/workflows/test-changes.yml    | 37 +++++++++++++++------------
 3 files changed, 30 insertions(+), 27 deletions(-)

diff --git a/.github/workflows/codacy-analysis.yml b/.github/workflows/codacy-analysis.yml
index d1e612b0..546b2958 100644
--- a/.github/workflows/codacy-analysis.yml
+++ b/.github/workflows/codacy-analysis.yml
@@ -16,11 +16,11 @@ name: Codacy Security Scan
 on:
   schedule:
     - cron: '59 11 27 * *'
-  # push:
-  #   branches: [ "master" ]
-  # pull_request:
-  #   branches: [ "master" ]
-  #   types: [opened, reopened, synchronize, ready_for_review]
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+    types: [opened, reopened, synchronize, ready_for_review]
   # workflow_run:
   #   workflows: [Test Changes]
   #   types:
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index 2db3c985..090b900f 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -14,11 +14,11 @@ name: "CodeQL"
 on:
   schedule:
     - cron: '59 10 27 * *'
-  # push:
-  #   branches: [ "master" ]
-  # pull_request:
-  #   branches: [ "master" ]
-  #   types: [opened, reopened, synchronize, ready_for_review]
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+    types: [opened, reopened, synchronize, ready_for_review]
 # workflow_run:
   #   workflows: [Test Changes]
   #   types:
diff --git a/.github/workflows/test-changes.yml b/.github/workflows/test-changes.yml
index baea6c53..2a3bfa82 100644
--- a/.github/workflows/test-changes.yml
+++ b/.github/workflows/test-changes.yml
@@ -202,21 +202,24 @@ jobs:
           cd docs
           sphinx-build -W -b singlehtml -d ../build/doctrees . ../build/singlehtml
 
-  call-workflow-codeql:
-    needs: test-source-code
-    uses: ./.github/workflows/codeql-analysis.yml
-    permissions:
-      actions: read
-      contents: read
-      pull-requests: write
-      security-events: write
-
-  call-workflow-codacity:
-    needs: test-source-code
-    uses: ./.github/workflows/codacy-analysis.yml
-    permissions:
-      actions: read
-      contents: read
-      pull-requests: write
-      security-events: write
+  # TODO: 1 configuration not found: https://docs.github.com/en/actions/using-workflows/reusing-workflows#overview
+  # call-workflow-codeql:
+  #   needs: test-source-code
+  #   uses: ./.github/workflows/codeql-analysis.yml
+  #   permissions:
+  #     actions: read
+  #     contents: read
+  #     pull-requests: write
+  #     security-events: write
+  #   secrets: inherit # pass all secrets
+
+  # call-workflow-codacity:
+  #   needs: test-source-code
+  #   uses: ./.github/workflows/codacy-analysis.yml
+  #   permissions:
+  #     actions: read
+  #     contents: read
+  #     pull-requests: write
+  #     security-events: write
+  #   secrets: inherit # pass all secrets