From b14e0c84e2d19ed80f84d01c17ef4c4345047edd Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Tue, 24 Sep 2024 14:42:17 +0530
Subject: [PATCH] [pre-commit.ci] pre-commit autoupdate (#503)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.5 → v0.6.7](https://github.com/astral-sh/ruff-pre-commit/compare/v0.6.5...v0.6.7)
- [github.com/PyCQA/pylint: v3.2.7 → v3.3.0](https://github.com/PyCQA/pylint/compare/v3.2.7...v3.3.0)

* Ignore too-many-positional-arguments

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kiran Jonnalagadda <kiran@hasgeek.com>
---
 .pre-commit-config.yaml |  4 ++--
 pyproject.toml          | 27 ++++++++++++++-------------
 2 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 56176888..61bbbc87 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -8,7 +8,7 @@ repos:
     hooks:
       - id: check-pre-commit-ci-config
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.6.5
+    rev: v0.6.7
     hooks:
       - id: ruff
         args: ['--fix', '--exit-non-zero-on-fix']
@@ -71,7 +71,7 @@ repos:
       - id: flake8
         additional_dependencies: *flake8deps
   - repo: https://github.com/PyCQA/pylint
-    rev: v3.2.7
+    rev: v3.3.0
     hooks:
       - id: pylint
         args: [
diff --git a/pyproject.toml b/pyproject.toml
index cd9a00bd..678f3f9f 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -144,26 +144,27 @@ for path in pathlib.Path.cwd().parents:
 [tool.pylint.message_control]
 max-line-length = 88
 disable = [
-  'cyclic-import',                # We have tail imports all over
-  'fixme',                        # Our workflow is to tag for future fixes
-  'invalid-name',                 # Flake8 covers our naming convention requirements
-  'line-too-long',                # Let Black/Ruff handle this
-  'missing-class-docstring',      # Backlog
-  'missing-function-docstring',   # Backlog
-  'no-member',                    # Pylint doesn't understand mixins referring to main class members
-  'redefined-builtin',            # Covered by Flake8 already
-  'too-few-public-methods',       # Data classes and validator classes have few methods
+  'cyclic-import',                 # We have tail imports all over
+  'fixme',                         # Our workflow is to tag for future fixes
+  'invalid-name',                  # Flake8 covers our naming convention requirements
+  'line-too-long',                 # Let Black/Ruff handle this
+  'missing-class-docstring',       # Backlog
+  'missing-function-docstring',    # Backlog
+  'no-member',                     # Pylint doesn't understand mixins referring to main class members
+  'redefined-builtin',             # Covered by Flake8 already
+  'too-few-public-methods',        # Data classes and validator classes have few methods
   'too-many-arguments',
   'too-many-branches',
   'too-many-locals',
-  'too-many-instance-attributes', # Some instances are just bags of attributes
+  'too-many-instance-attributes',  # Some instances are just bags of attributes
   'too-many-lines',
   'too-many-nested-blocks',
+  'too-many-positional-arguments',
   'too-many-return-statements',
   'too-many-statements',
-  'unused-argument',              # Arguments required for spec compatibility aren't always used
-  'wrong-import-position',        # Imports after code are sometimes required
-  'wrong-import-order',           # Let black and isort handle this
+  'unused-argument',               # Arguments required for spec compatibility aren't always used
+  'wrong-import-position',         # Imports after code are sometimes required
+  'wrong-import-order',            # Let black and isort handle this
 ]
 
 [tool.bandit]