add code spell and pygrep-hooks

Author: Vahid Tavanashad

.github/workflows/conda-package-cf.yml

@@ -301,8 +301,8 @@ jobs:
       - name: Report content of test environment
         shell: cmd /C CALL {0}
         run: |
-          echo "Value of CONDA enviroment variable was: " %CONDA%
-          echo "Value of CONDA_PREFIX enviroment variable was: " %CONDA_PREFIX%
+          echo "Value of CONDA environment variable was: " %CONDA%
+          echo "Value of CONDA_PREFIX environment variable was: " %CONDA_PREFIX%
           conda info && conda list -n ${{ env.TEST_ENV_NAME }}
 
       - name: Run tests

.github/workflows/conda-package.yml

@@ -301,8 +301,8 @@ jobs:
       - name: Report content of test environment
         shell: cmd /C CALL {0}
         run: |
-          echo "Value of CONDA enviroment variable was: " %CONDA%
-          echo "Value of CONDA_PREFIX enviroment variable was: " %CONDA_PREFIX%
+          echo "Value of CONDA environment variable was: " %CONDA%
+          echo "Value of CONDA_PREFIX environment variable was: " %CONDA_PREFIX%
           conda info && conda list -n ${{ env.TEST_ENV_NAME }}
       - name: Run tests
         shell: cmd /C CALL {0}

.github/workflows/pre-commit.yml

@@ -20,13 +20,18 @@ jobs:
       with:
         python-version: '3.12'
 
-    - uses: BSFishy/pip-action@v1
+    - name: Set up pip packages
+      uses: BSFishy/pip-action@v1
       with:
         packages: |
+          codespell
           pylint
 
-    - name: Version of clang-format
+    - name: Set up clang-format
       run: |
+        sudo apt-get install -y clang-format-14
+        sudo unlink /usr/bin/clang-format
+        sudo ln -s /usr/bin/clang-format-14 /usr/bin/clang-format
         clang-format --version
 
     - name: Run pre-commit checks

.pre-commit-config.yaml

@@ -15,6 +15,28 @@ repos:
     - id: mixed-line-ending
     - id: trailing-whitespace
 
+-   repo: https://github.com/pre-commit/pygrep-hooks
+    rev: v1.10.0
+    hooks:
+    -   id: python-check-blanket-noqa
+    -   id: python-check-blanket-type-ignore
+    -   id: python-check-mock-methods
+    -   id: python-no-eval
+    -   id: python-no-log-warn
+    -   id: python-use-type-annotations
+    -   id: rst-backticks
+    -   id: rst-directive-colons
+    -   id: rst-inline-touching-normal
+    -   id: text-unicode-replacement-char
+
+- repo: https://github.com/codespell-project/codespell
+  rev: v2.4.1
+  hooks:
+  - id: codespell
+    args: ["-L", "nd"]  # ignore "nd" used for n-dimensional
+    additional_dependencies:
+    - tomli
+
 - repo: https://github.com/psf/black
   rev: 25.1.0
   hooks:

mkl_fft/_scipy_fft.py

@@ -43,7 +43,7 @@
 from ._float_utils import _supported_array_or_not_implemented
 
 __doc__ = """
-This module implements interfaces mimicing `scipy.fft` module.
+This module implements interfaces mimicking `scipy.fft` module.
 
 It also provides DftiBackend class which can be used to set mkl_fft to be used
 via `scipy.fft` namespace.

mkl_fft/src/mklfft.c.src

@@ -363,7 +363,7 @@ compute_strides_and_distances(
         break;
     default:
         /* we must have C- or F- contiguous layout */
-        /* TODO: enhance the test to acommodate equal size slices along all axis of a contiguous array,  *
+        /* TODO: enhance the test to accommodate equal size slices along all axis of a contiguous array,  *
          *       for example x = contig_r3_arr[::4, ::4, ::4]                                            */
         {
             int any_contig = (PyArray_ISONESEGMENT(x)) ? 1 : 0;

mkl_fft/tests/third_party/numpy/test_fft.py

@@ -175,7 +175,7 @@ def test_fft_inplace_out(self, axis):
         else:
             assert_array_equal(y3[:, :5], y[:, :5])
             assert_array_equal(y3[:, 15:], y[:, 15:])
-        # In-place with n > nin; rest should be unchanged.
+        # In-place with n > n_in; rest should be unchanged.
         y4 = y.copy()
         y4_sel = y4[:10] if axis == 0 else y4[:, :10]
         out4 = y4[:15] if axis == 0 else y4[:, :15]