Repository: ocean-eddy-cpt/gcm-filters (language: python)

Issue:
Failing tests and code formatting
Both the actions for pre-commit and the tests show failed status on the last master branch execution. 

I am not able to see the logs anymore (it was over a month ago). 

I suggest to set up a weekly run of the CI, even if no changes were submitted. This would avoid any problems with transient failures and display a more accurate state of the repo.




Discussion / hints:

Fix patch:
diff --git a/.github/workflows/pre-commit.yml b/.github/workflows/pre-commit.yml
index 47a84b2..0312d87 100644
--- a/.github/workflows/pre-commit.yml
+++ b/.github/workflows/pre-commit.yml
@@ -1,9 +1,14 @@
 name: pre-commit
 
 on:
-  pull_request:
   push:
-    branches: [master]
+    branches:
+      - "master"
+  pull_request:
+    branches:
+      - "*"
+  schedule:
+    - cron: "0 13 * * 1"
 
 jobs:
   pre-commit:


Test patch:
diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
index acd6d06..1d8c5ac 100644
--- a/.github/workflows/tests.yaml
+++ b/.github/workflows/tests.yaml
@@ -1,5 +1,13 @@
 name: Tests
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - "master"
+  pull_request:
+    branches:
+      - "*"
+  schedule:
+    - cron: "0 13 * * 1"
 
 jobs:
   build:
diff --git a/tests/test_kernels.py b/tests/test_kernels.py
index 76efb7e..7ea2ef9 100644
--- a/tests/test_kernels.py
+++ b/tests/test_kernels.py
@@ -178,7 +178,7 @@ def test_flux_in_x_direction(grid_type_field_and_extra_kwargs):
 
 
 def test_for_antarctica(grid_type_field_and_extra_kwargs):
-    """This test checks that we get an error if southernmost row of wet_mask has entry not equal to zero. """
+    """This test checks that we get an error if southernmost row of wet_mask has entry not equal to zero."""
     grid_type, _, extra_kwargs = grid_type_field_and_extra_kwargs
 
     if grid_type in tripolar_grids:
@@ -193,7 +193,7 @@ def test_for_antarctica(grid_type_field_and_extra_kwargs):
 
 
 def test_tripolar_exchanges(grid_type_field_and_extra_kwargs):
-    """This test checks that Laplacian exchanges across northern boundary seam line of tripolar grid are correct. """
+    """This test checks that Laplacian exchanges across northern boundary seam line of tripolar grid are correct."""
     grid_type, data, extra_kwargs = grid_type_field_and_extra_kwargs
 
     if grid_type in tripolar_grids: