pre-commit: Upgrade hook versions

Signed-off-by: Marcel Lauhoff <[email protected]>
s3gw-tech · Nov 13, 2023 · c28d57d · c28d57d
1 parent 4d2b9f1
commit c28d57d
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 13 deletions.
diff --git a/.github/workflows/nightly.yaml b/.github/workflows/nightly.yaml
@@ -149,7 +149,7 @@ jobs:
           export S3TEST_LIST="${FIXTURES}/s3-tests.txt"
 
           # There are some s3tests that don't finish at all. Only run
-          # known-to-pass tests for now. TODO: fix inifite looping tests
+          # known-to-pass tests for now. TODO: fix infinite looping tests
           # sed -r -i 's/^# //' "${S3TEST_LIST}"
 
           pushd s3tests

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,7 +1,7 @@
 ---
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.0.1
+    rev: v4.5.0
     hooks:
       - id: trailing-whitespace
       - id: check-yaml
@@ -17,7 +17,7 @@ repos:
       - id: check-case-conflict
 
   - repo: https://github.com/jumanjihouse/pre-commit-hooks
-    rev: 2.1.5
+    rev: 3.0.0
     hooks:
       - id: forbid-binary
         exclude: >-
@@ -27,12 +27,12 @@ repos:
       - id: git-check
 
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.1.0
+    rev: v2.2.6
     hooks:
       - id: codespell
 
   - repo: https://github.com/streetsidesoftware/cspell-cli
-    rev: v6.14.1
+    rev: v8.0.0
     hooks:
     - id: cspell
       args:
@@ -49,7 +49,7 @@ repos:
       - id: markdownlint-cli2
 
   - repo: https://github.com/ambv/black
-    rev: 23.3.0
+    rev: 23.11.0
     hooks:
     - id: black
       language_version: python3.10
@@ -62,7 +62,7 @@ repos:
         files: (?x)^tools/(release|s3tests)/.+\.py$
 
   - repo: https://github.com/pycqa/flake8
-    rev: 4.0.1
+    rev: 6.1.0
     hooks:
       - id: flake8
         additional_dependencies:
@@ -72,7 +72,7 @@ repos:
         files: (?x)^tools/(release|s3tests)/.+\.py$
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: 'v1.1.1'
+    rev: 'v1.7.0'
     hooks:
       - id: mypy
         additional_dependencies:
@@ -82,7 +82,7 @@ repos:
         files: (?x)^tools/(release|s3tests)/.+\.py$
 
   - repo: https://github.com/asottile/pyupgrade
-    rev: v2.10.0
+    rev: v3.15.0
     hooks:
       - id: pyupgrade
         args: [--py36-plus]

diff --git a/tools/tests/on-disk-format-checker.sh b/tools/tests/on-disk-format-checker.sh
@@ -84,11 +84,10 @@ start_s3gw() {
     echo "Container failed to start or crashed"
     echo "Showing logs..."
     echo "---------------------------------------------"
-    _podman logs ${CONTAINER}
+    _podman logs "${CONTAINER}"
     echo "---------------------------------------------"
-    METADATA_ISSUE=$(_podman logs ${CONTAINER} | grep "ERROR ACCESSING SFS METADATA")
-    if [ "$?" -eq 0 ]; then
-      echo "Format of metadata has changed. Breaking changes inconsitencies found."
+    if _podman logs "${CONTAINER}" | grep "ERROR ACCESSING SFS METADATA"; then
+      echo "Format of metadata has changed. Breaking changes inconsistencies found."
     fi
     exit 1
   fi