Add ruff gh action

.github/workflows/run-tests.yml

@@ -18,6 +18,7 @@ jobs:
 
     steps:
     - uses: actions/checkout@v2
+    - uses: chartboost/ruff-action@v1
     - name: Set up Python ${{ matrix.python-version }}
       uses: actions/setup-python@v2
       with:

.pre-commit-config.yaml

@@ -0,0 +1,7 @@
+repos:
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.3.3
+    hooks:
+      - id: ruff
+        args: ["--fix"]
+      - id: ruff-format

Makefile

@@ -15,4 +15,7 @@ release:
 	@git push --follow-tags --all
 	@git push --tags
 
-.PHONY: test tag coverage clean release
+install-hooks:
+	@pre-commit install -f --install-hooks -t pre-commit
+
+.PHONY: test tag coverage clean release install-hooks

ruff.toml

@@ -1,2 +1,5 @@
 line-length = 121
 indent-width = 4
+
+[lint]
+ignore = ["F401", "F403"]

yamale/schema/schema.py

@@ -172,8 +172,8 @@ def _validate_any(self, validator, data, path, strict):
     def _validate_subset(self, validator, data, path, strict):
         def _internal_validate(internal_data):
             sub_errors = []
-            for val in validator.validators:
-                err = self._validate(val, internal_data, path, strict)
+            for v in validator.validators:
+                err = self._validate(v, internal_data, path, strict)
                 if not err:
                     break
                 sub_errors += err

yamale/util.py

@@ -1,27 +1,12 @@
-# ABCs for containers were moved to their own module
-try:
-    from collections.abc import Mapping, Set, Sequence
-except ImportError:
-    from collections import Mapping, Set, Sequence
+from collections.abc import Mapping, Sequence
 
 
-# Python 3 has no basestring, lets test it.
-try:
-    basestring  # attempt to evaluate basestring
+def isstr(s):
+    return isinstance(s, str)
 
-    def isstr(s):
-        return isinstance(s, basestring)
 
-    def to_unicode(s):
-        return unicode(s)
-
-except NameError:
-
-    def isstr(s):
-        return isinstance(s, str)
-
-    def to_unicode(s):
-        return s
+def to_unicode(s):
+    return s
 
 
 def is_list(obj):

yamale/yamale_testcase.py

@@ -26,7 +26,7 @@ def validate(self, validators=None):
         if schema is None:
             return
 
-        if type(yaml) != list:
+        if not isinstance(yaml, list):
             yaml = [yaml]
 
         if base_dir is not None: