From 3bb5b4a3025dc2081992b853c6363c7c20c50082 Mon Sep 17 00:00:00 2001
From: Nripesh Niketan <86844847+NripeshN@users.noreply.github.com>
Date: Fri, 6 Dec 2024 15:54:29 +0000
Subject: [PATCH] Chore: Add default language in pre-commit and bump hooks
 (#1652)

---
 .pre-commit-config.yaml |  8 ++++++--
 tests/array_tests.cpp   | 44 ++++++++++++++++++++++++++++++-----------
 tests/ops_tests.cpp     |  8 ++++++--
 3 files changed, 45 insertions(+), 15 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index c2b6f1f34..bd3d21d70 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -1,13 +1,17 @@
+default_language_version:
+  python: python3.10
+
 repos:
 -   repo: https://github.com/pre-commit/mirrors-clang-format
-    rev: v18.1.8
+    rev: v19.1.4
     hooks:
     -   id: clang-format
 # Using this mirror lets us use mypyc-compiled black, which is about 2x faster
 -   repo: https://github.com/psf/black-pre-commit-mirror
-    rev: 24.8.0
+    rev: 24.10.0
     hooks:
     -   id: black
+    
 -   repo: https://github.com/pycqa/isort
     rev: 5.13.2
     hooks:
diff --git a/tests/array_tests.cpp b/tests/array_tests.cpp
index 5144243e7..f91c0a313 100644
--- a/tests/array_tests.cpp
+++ b/tests/array_tests.cpp
@@ -129,37 +129,59 @@ TEST_CASE("test array types") {
   }
 
   // uint8
-  { basic_dtype_test(uint8_t, uint8); }
+  {
+    basic_dtype_test(uint8_t, uint8);
+  }
 
   // uint16
-  { basic_dtype_test(uint16_t, uint16); }
+  {
+    basic_dtype_test(uint16_t, uint16);
+  }
 
   // uint32
-  { basic_dtype_test(uint32_t, uint32); }
+  {
+    basic_dtype_test(uint32_t, uint32);
+  }
 
   // uint64
-  { basic_dtype_test(uint64_t, uint64); }
+  {
+    basic_dtype_test(uint64_t, uint64);
+  }
 
   // int8
-  { basic_dtype_test(int8_t, int8); }
+  {
+    basic_dtype_test(int8_t, int8);
+  }
 
   // int16
-  { basic_dtype_test(int16_t, int16); }
+  {
+    basic_dtype_test(int16_t, int16);
+  }
 
   // int32
-  { basic_dtype_test(int32_t, int32); }
+  {
+    basic_dtype_test(int32_t, int32);
+  }
 
   // int64
-  { basic_dtype_test(int64_t, int64); }
+  {
+    basic_dtype_test(int64_t, int64);
+  }
 
   // float16
-  { basic_dtype_test(float16_t, float16); }
+  {
+    basic_dtype_test(float16_t, float16);
+  }
 
   // float32
-  { basic_dtype_test(float, float32); }
+  {
+    basic_dtype_test(float, float32);
+  }
 
   // bfloat16
-  { basic_dtype_test(bfloat16_t, bfloat16); }
+  {
+    basic_dtype_test(bfloat16_t, bfloat16);
+  }
 
 #undef basic_dtype_test
 
diff --git a/tests/ops_tests.cpp b/tests/ops_tests.cpp
index 4b32d5794..9d87b8e2c 100644
--- a/tests/ops_tests.cpp
+++ b/tests/ops_tests.cpp
@@ -3045,13 +3045,17 @@ TEST_CASE("test divmod") {
 
   // Check that we can still eval when one output goes out of scope
   std::vector<array> out_holder;
-  { out_holder.push_back(divmod(x, y)[0]); }
+  {
+    out_holder.push_back(divmod(x, y)[0]);
+  }
   eval(out_holder);
   CHECK_EQ(out_holder[0].item<float>(), 0.0);
 
   // Check that we can still eval when the other output goes out of scope
   out_holder.clear();
-  { out_holder.push_back(divmod(x, y)[1]); }
+  {
+    out_holder.push_back(divmod(x, y)[1]);
+  }
   eval(out_holder);
   CHECK_EQ(out_holder[0].item<float>(), 1.0);
 }