From 4b36f6a333828b0af575090fb75497ddbbfdf1bb Mon Sep 17 00:00:00 2001
From: Carlos Camacho <carloscamachoucv@gmail.com>
Date: Fri, 30 Jun 2023 09:05:25 +0200
Subject: [PATCH] False positive output in tox linter checks

This commit:
- Makes easier to test individually each commit by allowing
to run the GitHub actions on each fork push and pull request.
- Remove the appended path in the ruff tox check that breaks it
to parse recursivelly all the python code.
- Excluding the test and tests folders on the ruff check to make
easier to pass the functional code linter checks

TODO: This should be applied to all the other tox check to make them
show correctly all the linters errors. And once the functional code
is green, to make pass the tests code.

Closes: #145
---
 .github/workflows/integration-tests.yml | 2 --
 .github/workflows/linters.yml           | 2 --
 .github/workflows/tests.yml             | 2 --
 .github/workflows/tox.ini               | 6 ++----
 .github/workflows/tox.yml               | 6 ++----
 5 files changed, 4 insertions(+), 14 deletions(-)

diff --git a/.github/workflows/integration-tests.yml b/.github/workflows/integration-tests.yml
index 05053994..022ff6ff 100644
--- a/.github/workflows/integration-tests.yml
+++ b/.github/workflows/integration-tests.yml
@@ -1,9 +1,7 @@
 name: Run integration tests
 on:
   push:
-    branches: ["main"]
   pull_request:
-    branches: ["main"]
   # Run the integration tests every 8 hours.
   # This will help to identify faster if
   # there is a CI failure related to a
diff --git a/.github/workflows/linters.yml b/.github/workflows/linters.yml
index 8c82f904..0e365341 100644
--- a/.github/workflows/linters.yml
+++ b/.github/workflows/linters.yml
@@ -2,9 +2,7 @@
 name: Run linters
 on:
   push:
-    branches: ["main"]
   pull_request:
-    branches: ["main"]
   # Run the linters tests every 8 hours.
   # This will help to identify faster if
   # there is a CI failure related to a
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 6c55bc62..7606d7c0 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -1,9 +1,7 @@
 name: Run Ansible tests
 on:
   push:
-    branches: ["main"]
   pull_request:
-    branches: ["main"]
   # Run the unit tests every 8 hours.
   # This will help to identify faster if
   # there is a CI failure related to a
diff --git a/.github/workflows/tox.ini b/.github/workflows/tox.ini
index 1c6ad867..d0d0305f 100644
--- a/.github/workflows/tox.ini
+++ b/.github/workflows/tox.ini
@@ -12,13 +12,11 @@ requires =
 
 [testenv:ruff]
 deps = ruff
-commands = ruff check --select ALL --ignore INP001 -q {posargs}/extensions/eda/plugins
-
+commands = ruff check --select ALL --ignore INP001 --extend-exclude test,tests -q {posargs}
 
 [testenv:darglint]
 deps = darglint
-commands = darglint -s numpy -z full {posargs}/extensions/eda/plugins
-
+commands = darglint -s numpy -z full {posargs}
 
 # If you dont have any event_source or event_filter plugins, remove the corresponding testenv
 [testenv:pylint-event-source]
diff --git a/.github/workflows/tox.yml b/.github/workflows/tox.yml
index 348bd0e9..2b8bce10 100644
--- a/.github/workflows/tox.yml
+++ b/.github/workflows/tox.yml
@@ -1,10 +1,8 @@
 ---
-name: Tox
+name: Run Tox linters checks
 on:
   push:
-    branches: ["main"]
   pull_request:
-    branches: ["main"]
   # Run the tox tests every 8 hours.
   # This will help to identify faster if
   # there is a CI failure related to a
@@ -21,4 +19,4 @@ jobs:
       - name: Move to tox conf file and run tox
         run: | 
           cd .github/workflows
-          python -m tox -- ../..
+          python3 -m tox -- ../..