Revert "Temporary remove custom pylint check (#3718)" (#3787)

Also bump up pylint version to 3.3.6, which has the original issue
fixed.

Author: pbchekin

.pre-commit-config.yaml

@@ -59,6 +59,66 @@ repos:
       files: ^.github/workflows/integration-tests.yml.*
       pass_filenames: false
 
+  - repo: https://github.com/pycqa/pylint
+    rev: v3.3.6
+    hooks:
+      - id: pylint
+        name: pylint for scripts
+        files: '^scripts/.*\.py$'
+        args:
+          - --check-quote-consistency=true
+          # Max line length 120 is consistent with ruff's one in upstream
+          - --max-line-length=120
+          # Disable line-too-long: rely on ruff's check for line length
+          - --disable=line-too-long
+          # Disable import-error: not everything can be imported when pre-commit runs
+          - --disable=import-error
+          # Disable unused-import: ruff has a corresponding check and supports "noqa: F401"
+          - --disable=unused-import
+          # FIXME: revisit these checks later
+          - --disable=missing-module-docstring
+          - --disable=missing-function-docstring
+          - --disable=missing-class-docstring
+          - --disable=duplicate-code
+          - --disable=too-many-locals
+          - --disable=too-many-statements
+          - --disable=too-many-arguments
+          - --disable=too-many-positional-arguments
+        stages: [pre-commit, pre-push, manual]
+
+      - id: pylint
+        name: pylint for benchmarks
+        files: '^benchmarks/.*\.py$'
+        args:
+          - --check-quote-consistency=true
+          # Max line length 120 is consistent with ruff's one in upstream
+          - --max-line-length=120
+          # Disable line-too-long: rely on ruff's check for line length
+          - --disable=line-too-long
+          # Disable import-error: not everything can be imported when pre-commit runs
+          - --disable=import-error
+          - --disable=no-name-in-module
+          # Disable unused-import: ruff has a corresponding check and supports "noqa: F401"
+          - --disable=unused-import
+          # Disable invalid_name: benchmarks use a lot of UPPER_SNAKE arguments
+          - --disable=invalid-name
+          # Disable unnecessary-lambda-assignment: benchmark use a lot of lambda assignments
+          - --disable=unnecessary-lambda-assignment
+          # FIXME: revisit these checks later
+          - --disable=import-outside-toplevel
+          - --disable=too-few-public-methods
+          - --disable=consider-using-generator
+          - --disable=missing-module-docstring
+          - --disable=missing-function-docstring
+          - --disable=missing-class-docstring
+          - --disable=duplicate-code
+          - --disable=too-many-locals
+          - --disable=too-many-statements
+          - --disable=too-many-arguments
+          - --disable=too-many-positional-arguments
+          - --disable=fixme
+        stages: [pre-commit, pre-push, manual]
+
   - repo: https://github.com/pre-commit/mirrors-mypy
     rev: v1.13.0
     hooks: