From 5106dc3e73d0b55f8eb0bfb8375c103c93402c1a Mon Sep 17 00:00:00 2001
From: Madison Swain-Bowden <bowdenm@spu.edu>
Date: Wed, 5 Apr 2023 15:14:55 -0700
Subject: [PATCH] Update pre-commit deps, pin actionlint (#1139)

---
 .pre-commit-config.yaml                 | 16 ++++++++--------
 automations/python/issues_with_prs.py   |  2 +-
 documentation/_ext/link_issues.py       |  1 -
 templates/.pre-commit-config.yaml.jinja | 16 ++++++++--------
 4 files changed, 17 insertions(+), 18 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index f66b355ac21..cdef9f5def2 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -18,7 +18,7 @@ repos:
         pass_filenames: false
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.0.1
+    rev: v4.4.0
     hooks:
       - id: trailing-whitespace
         args: [--markdown-linebreak-ext=md]
@@ -52,26 +52,26 @@ repos:
           - --lines-after-imports=2
 
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.2.2
+    rev: v3.3.1
     hooks:
       - id: pyupgrade
         args:
           - --py310-plus
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: "v0.0.257"
+    rev: "v0.0.261"
     hooks:
       - id: ruff
 
   - repo: https://github.com/ambv/black
-    rev: 22.3.0
+    rev: 23.3.0
     hooks:
       - id: black
         args:
           - --safe
 
   - repo: https://github.com/pycqa/pydocstyle
-    rev: 6.2.2 # 6.2.3 is slightly broken
+    rev: 6.3.0
     hooks:
       - id: pydocstyle
         args:
@@ -87,7 +87,7 @@ repos:
           - --add-ignore=D1,D202,D205,D211,D400
 
   - repo: https://github.com/pre-commit/mirrors-eslint
-    rev: v8.15.0
+    rev: v8.37.0
     hooks:
       - id: eslint
         files: \.(js|ts|vue)$
@@ -125,12 +125,12 @@ repos:
           - typescript@4.9.3
 
   - repo: https://github.com/koalaman/shellcheck-precommit
-    rev: v0.8.0
+    rev: v0.9.0
     hooks:
       - id: shellcheck
 
   - repo: https://github.com/rhysd/actionlint
-    rev: main
+    rev: v1.6.24
     hooks:
       - id: actionlint-docker
 
diff --git a/automations/python/issues_with_prs.py b/automations/python/issues_with_prs.py
index 91c29a5a5b2..39838d1021d 100755
--- a/automations/python/issues_with_prs.py
+++ b/automations/python/issues_with_prs.py
@@ -247,7 +247,7 @@ def main():
             cards_to_move.append((issue_card, issue))
     log.info(f"Found {len(cards_to_move)} cards to move")
 
-    for (issue_card, issue) in cards_to_move:
+    for issue_card, issue in cards_to_move:
         log.info(f"Moving card for issue {issue.html_url} to {target_column.name}")
         issue_card.move("bottom", target_column)
 
diff --git a/documentation/_ext/link_issues.py b/documentation/_ext/link_issues.py
index b2e7958bc1e..fa82d1af8fb 100644
--- a/documentation/_ext/link_issues.py
+++ b/documentation/_ext/link_issues.py
@@ -89,7 +89,6 @@ class PendingIssueXRef(pending_xref):
 
 
 class IssueReferences(SphinxTransform):
-
     default_priority = 999
 
     def apply(self) -> None:
diff --git a/templates/.pre-commit-config.yaml.jinja b/templates/.pre-commit-config.yaml.jinja
index 2b3f6170894..a19a50b941c 100644
--- a/templates/.pre-commit-config.yaml.jinja
+++ b/templates/.pre-commit-config.yaml.jinja
@@ -11,7 +11,7 @@ repos:
   {% block repos_top %}
   {% endblock %}
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.0.1
+    rev: v4.4.0
     hooks:
       - id: trailing-whitespace
         args: [--markdown-linebreak-ext=md]
@@ -48,26 +48,26 @@ repos:
           - --lines-after-imports=2
 
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.2.2
+    rev: v3.3.1
     hooks:
       - id: pyupgrade
         args:
           - --py310-plus
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: "v0.0.257"
+    rev: "v0.0.261"
     hooks:
       - id: ruff
 
   - repo: https://github.com/ambv/black
-    rev: 22.3.0
+    rev: 23.3.0
     hooks:
       - id: black
         args:
           - --safe
 
   - repo: https://github.com/pycqa/pydocstyle
-    rev: 6.2.2 # 6.2.3 is slightly broken
+    rev: 6.3.0
     hooks:
       - id: pydocstyle
         args:
@@ -85,7 +85,7 @@ repos:
   {% endif %}
   {% if contains_js_code | default(true) %}
   - repo: https://github.com/pre-commit/mirrors-eslint
-    rev: v8.15.0
+    rev: v8.37.0
     hooks:
       - id: eslint
         files: {{ eslint_files | default("^js/.*$") }}
@@ -117,12 +117,12 @@ repos:
 
 {# This comment adds a blank line. #}
   - repo: https://github.com/koalaman/shellcheck-precommit
-    rev: v0.8.0
+    rev: v0.9.0
     hooks:
       - id: shellcheck
 
   - repo: https://github.com/rhysd/actionlint
-    rev: main
+    rev: v1.6.24
     hooks:
       - id: actionlint-docker
   {% block repos_bottom %}