micropython · cclauss · Feb 18, 2025
diff --git a/.github/workflows/precommit.yml b/.github/workflows/precommit.yml
@@ -0,0 +1,24 @@
+name: pre-commit
+
+on: [push, pull_request]
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  hook:
+    strategy:
+        fail-fast: false
+        matrix:
+          # codeformat generates too many changes
+          # ruff-format is blocked by micropython/micropython#16774
+          # hook: [codeformat, codespell, ruff, ruff-format]
+          hook: [codespell, ruff]
+    runs-on: ubuntu-22.04
+    steps:
+    - uses: actions/checkout@v4
+    - uses: actions/setup-python@v5
+    - uses: pre-commit/action@v3.0.1
+      with:
+        extra_args: ${{ matrix.hook }} --all-files
diff --git a/pyproject.toml b/pyproject.toml
@@ -5,18 +5,18 @@ ignore-words-list = "ans,asend,deques,dout,emac,extint,hsi,iput,mis,numer,shft,s
 quiet-level = 3
 skip = """
 */build*,\
-./.git,\
-./drivers/cc3100,\
-./lib,\
-./ports/cc3200/FreeRTOS,\
-./ports/cc3200/bootmgr/sl,\
-./ports/cc3200/hal,\
-./ports/cc3200/simplelink,\
-./ports/cc3200/telnet,\
-./ports/esp32/managed_components,\
+./.git/*,\
+./drivers/cc3100/*,\
+./lib/*,\
+./ports/cc3200/FreeRTOS/*,\
+./ports/cc3200/bootmgr/sl/*,\
+./ports/cc3200/hal/*,\
+./ports/cc3200/simplelink/*,\
+./ports/cc3200/telnet/*,\
+./ports/esp32/managed_components/*,\
 ./ports/nrf/drivers/bluetooth/s1*,\
-./ports/stm32/usbhost,\
-./tests,\
+./ports/stm32/usbhost/*,\
+./tests/*,\
 ACKNOWLEDGEMENTS,\
 """
 
@@ -27,6 +27,17 @@ line-length = 99
 target-version = "py37"
 
 [tool.ruff.lint]
+exclude = [  # Ruff finds Python SyntaxError in these files
+  "tests/cmdline/repl_autocomplete.py",
+  "tests/cmdline/repl_autoindent.py",
+  "tests/cmdline/repl_basic.py",
+  "tests/cmdline/repl_cont.py",
+  "tests/cmdline/repl_emacs_keys.py",
+  "tests/cmdline/repl_words_move.py",
+  "tests/feature_check/repl_emacs_check.py",
+  "tests/feature_check/repl_words_move_check.py",
+  "tests/micropython/viper_args.py",
+]
 extend-select = ["C9", "PLC"]
 ignore = [
   "E401",

diff --git a/tests/README.md b/tests/README.md
@@ -13,7 +13,7 @@ target platform and run the appropriate set of tests for that platform.  For exa
 
 That will run tests on the `/dev/ttyACM0` serial port.  You can also use shortcut
 device names like `a<n>` for `/dev/ttyACM<n>` and `c<n>` for `COM<n>`.  Use
-`./run-tests.py --help` to see all of the device possibilites, and other options.
+`./run-tests.py --help` to see all of the device possibilities, and other options.
 
 There are three kinds of tests:
 

diff --git a/tests/extmod_hardware/machine_pwm.py b/tests/extmod_hardware/machine_pwm.py
@@ -1,4 +1,4 @@
-# Test machine.PWM, frequncy and duty cycle (using machine.time_pulse_us).
+# Test machine.PWM, frequency and duty cycle (using machine.time_pulse_us).
 #
 # IMPORTANT: This test requires hardware connections: the PWM-output and pulse-input
 # pins must be wired together (see the variable `pwm_pulse_pins`).

diff --git a/tests/ports/esp32/esp32_nvs.py b/tests/ports/esp32/esp32_nvs.py
@@ -4,7 +4,7 @@
 
 nvs = NVS("mp-test")
 
-# test setting and gettin an integer kv
+# test setting and getting an integer kv
 nvs.set_i32("key1", 1234)
 print(nvs.get_i32("key1"))
 nvs.set_i32("key2", -503)

diff --git a/tests/run-tests.py b/tests/run-tests.py
@@ -848,7 +848,7 @@ def run_one_test(test_file):
         test_file_abspath = os.path.abspath(test_file).replace("\\", "/")
 
         if args.filters:
-            # Default verdict is the opposit of the first action
+            # Default verdict is the opposite of the first action
             verdict = "include" if args.filters[0][0] == "exclude" else "exclude"
             for action, pat in args.filters:
                 if pat.search(test_file):