Lint code with ruff

Author: StanFromIreland

.github/workflows/lint.yml

@@ -0,0 +1,24 @@
+name: Lint
+
+on: [push, pull_request, workflow_dispatch]
+
+permissions: {}
+
+env:
+  FORCE_COLOR: 1
+  RUFF_OUTPUT_FORMAT: github
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+
+    steps:
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        with:
+          persist-credentials: false
+      - uses: j178/prek-action@0bb87d7f00b0c99306c8bcb8b8beba1eb581c037 # v1.1.1

.pre-commit-config.yaml

@@ -0,0 +1,21 @@
+repos:
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: a27a2e47c7751b639d2b5badf0ef6ff11fee893f  # frozen: v0.15.4
+    hooks:
+      - id: ruff
+        exclude: ^corp-
+      - id: ruff-format
+        exclude: ^corp-
+
+
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: 3e8a8703264a2f4a69428a0aa4dcb512790b2c8c  # frozen: v6.0.0
+    hooks:
+      - id: check-case-conflict
+      - id: check-merge-conflict
+      - id: end-of-file-fixer
+        exclude: ^corp-
+      - id: end-of-file-fixer
+        exclude: ^corp-
+      - id: trailing-whitespace
+        exclude: ^corp-

.ruff.toml

@@ -0,0 +1,5 @@
+[lint]
+ignore = [
+    "E722",  # bare except
+    "F841",  # unused variable
+]

README.md

@@ -28,7 +28,7 @@ Components that are hosted in this repository:
 
 Components that are hosted elsewhere:
 
-* [Project configuration](https://github.com/google/oss-fuzz/tree/master/projects/python3-libraries): 
+* [Project configuration](https://github.com/google/oss-fuzz/tree/master/projects/python3-libraries):
   This component controls the OSS-Fuzz project definition, maintainers.
 * [Fuzzer image configuration](https://github.com/google/oss-fuzz/tree/master/projects/python3-libraries):
   `Dockerfile` and `build.sh` describe how the fuzzer

ast.py

@@ -1,11 +1,12 @@
 import ast
 import warnings
 
-warnings.filterwarnings('ignore')
+warnings.filterwarnings("ignore")
+
 
 def FuzzerRunOne(FuzzerInput):
     # Python documentation states:
-    # 
+    #
     # 'It is possible to crash the Python interpreter with a sufficiently
     # large/complex string due to stack depth limitations in Python’s AST
     # compiler.'

configparser.py

@@ -1,10 +1,10 @@
 import configparser
 
+
 def FuzzerRunOne(FuzzerInput):
     parser = configparser.ConfigParser(allow_no_value=True, strict=False)
     try:
         parser.read_string(FuzzerInput.decode("utf-8", "replace"))
         list(parser.items())
     except configparser.Error:
         return
-

csv.py

@@ -1,9 +1,10 @@
 import csv
 
+
 def FuzzerRunOne(FuzzerInput):
     lines = FuzzerInput.decode("utf-8", "replace").splitlines()
     try:
-        reader = csv.reader(lines, delimiter=',')
+        reader = csv.reader(lines, delimiter=",")
         for row in reader:
             pass
     except csv.Error:

decode.py

@@ -1,7 +1,7 @@
 def FuzzerRunOne(FuzzerInput):
-    l = int(len(FuzzerInput)/2)
-    A = FuzzerInput[:l]
-    B = FuzzerInput[l:].decode("utf-8", "replace").strip()
+    half = int(len(FuzzerInput) / 2)
+    A = FuzzerInput[:half]
+    B = FuzzerInput[half:].decode("utf-8", "replace").strip()
     try:
         A.decode(B)
     except SystemError:

difflib.py

@@ -1,17 +1,16 @@
 import difflib
 
+
 def FuzzerRunOne(FuzzerInput):
     # SequenceMatcher is quadratic time worst case (see Lib/difflib.py).
     # Cap input size to avoid timeouts on large inputs.
     if not (0 < len(FuzzerInput) < 4096):
         return
-    l = int(len(FuzzerInput)/2)
-    A = FuzzerInput[:l].decode("utf-8", "replace").splitlines()
-    B = FuzzerInput[l:].decode("utf-8", "replace").splitlines()
+    half = int(len(FuzzerInput) / 2)
+    A = FuzzerInput[:half].decode("utf-8", "replace").splitlines()
+    B = FuzzerInput[half:].decode("utf-8", "replace").splitlines()
     for x in difflib.unified_diff(A, B):
         pass
     for x in difflib.context_diff(A, B):
         pass
     difflib.HtmlDiff().make_file(A, B)
-
-

email.py

@@ -1,12 +1,11 @@
-from email.parser import BytesParser, Parser
-from email.policy import default, HTTP
+from email.parser import Parser
+from email.policy import HTTP
+
 
 def FuzzerRunOne(FuzzerInput):
     try:
         Parser(policy=HTTP).parsestr(FuzzerInput.decode("utf-8", "replace"))
-    #    #Parser(policy=default).parsestr(FuzzerInput.decode("utf-8", "replace"))
     except SystemError:
         raise
     except:
         pass
-