From 08a68cb3b301bbce6aef2ae00d488dfd521f7ad1 Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Tue, 15 Aug 2023 21:44:07 +0800
Subject: [PATCH] [pre-commit.ci] pre-commit autoupdate (#1305)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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

updates:
- [github.com/Lucas-C/pre-commit-hooks: v1.5.3 → v1.5.4](https://github.com/Lucas-C/pre-commit-hooks/compare/v1.5.3...v1.5.4)
- [github.com/astral-sh/ruff-pre-commit: v0.0.282 → v0.0.284](https://github.com/astral-sh/ruff-pre-commit/compare/v0.0.282...v0.0.284)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
---
 .pre-commit-config.yaml                               |  4 ++--
 astronomer/providers/google/cloud/hooks/bigquery.py   | 11 +++--------
 .../providers/snowflake/extractors/snowflake.py       |  4 ++--
 3 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index f97dce6d9..218927eb6 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -30,7 +30,7 @@ repos:
       - id: python-check-blanket-type-ignore
 
   - repo: https://github.com/Lucas-C/pre-commit-hooks
-    rev: v1.5.3
+    rev: v1.5.4
     hooks:
       - id: forbid-crlf
       - id: remove-crlf
@@ -55,7 +55,7 @@ repos:
         additional_dependencies: [black==22.3.0]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: 'v0.0.282'
+    rev: 'v0.0.284'
     hooks:
       - id: ruff
         args: ["--config", "./pyproject.toml"]
diff --git a/astronomer/providers/google/cloud/hooks/bigquery.py b/astronomer/providers/google/cloud/hooks/bigquery.py
index 7bb50f276..f4adedd9e 100644
--- a/astronomer/providers/google/cloud/hooks/bigquery.py
+++ b/astronomer/providers/google/cloud/hooks/bigquery.py
@@ -106,14 +106,9 @@ def value_check(
         tolerance_pct_str = str(tolerance * 100) + "%" if tolerance else None
 
         error_msg = (
-            "Test failed.\nPass value:{pass_value_conv}\n"
-            "Tolerance:{tolerance_pct_str}\n"
-            "Query:\n{sql}\nResults:\n{records!s}"
-        ).format(
-            pass_value_conv=pass_value_conv,
-            tolerance_pct_str=tolerance_pct_str,
-            sql=sql,
-            records=records,
+            f"Test failed.\nPass value:{pass_value_conv}\n"
+            f"Tolerance:{tolerance_pct_str}\n"
+            f"Query:\n{sql}\nResults:\n{records!s}"
         )
 
         if not is_numeric_value_check:
diff --git a/astronomer/providers/snowflake/extractors/snowflake.py b/astronomer/providers/snowflake/extractors/snowflake.py
index d8a18fc9e..e2ea31338 100644
--- a/astronomer/providers/snowflake/extractors/snowflake.py
+++ b/astronomer/providers/snowflake/extractors/snowflake.py
@@ -78,8 +78,8 @@ def extract(self) -> TaskMetadata:
             )
         elif len(query_ids) > 1:
             warnings_msg = (
-                "Found more than one query id for task {0}: {1} This might indicate that this task "
-                "might be better as multiple jobs".format(task_name, query_ids)
+                f"Found more than one query id for task {task_name}: {query_ids} This might indicate that this task "
+                "might be better as multiple jobs"
             )
             self.log.warning(warnings_msg)