Skip to content

Commit 9b7a186

Browse files
committed
fix: linting & test tweaks
1 parent cb1902f commit 9b7a186

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

51 files changed

+898
-609
lines changed

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ repos:
1717
- id: mixed-line-ending
1818
- id: trailing-whitespace
1919
- repo: https://github.com/charliermarsh/ruff-pre-commit
20-
rev: "v0.14.9"
20+
rev: "v0.14.10"
2121
hooks:
2222
- id: ruff
2323
args: ["--fix"]

docs/examples/usage/usage_query_builder_10.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ def test_example_10(tmp_path: Path) -> None:
2626
# start-example
2727
# Multiple value sets
2828
query = (
29-
sql.insert("users")
29+
sql
30+
.insert("users")
3031
.columns("name", "email")
3132
.values("Alice", "[email protected]")
3233
.values("Bob", "[email protected]")

docs/examples/usage/usage_query_builder_13.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ def test_example_13(tmp_path: Path) -> None:
3030
from sqlglot import exp
3131

3232
query = (
33-
sql.update("users")
33+
sql
34+
.update("users")
3435
.set("name", "New Name")
3536
.set("email", "[email protected]")
3637
.set("updated_at", exp.CurrentTimestamp())

docs/examples/usage/usage_query_builder_17.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@ def test_example_17(tmp_path: Path) -> None:
2222
# start-example
2323
# Create table
2424
query = (
25-
sql.create_table("users")
25+
sql
26+
.create_table("users")
2627
.if_not_exists()
2728
.column("id", "INTEGER PRIMARY KEY")
2829
.column("name", "TEXT NOT NULL")

docs/examples/usage/usage_query_builder_22.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@ def test_example_22(tmp_path: Path) -> None:
3131
cte = sql.select("user_id", "COUNT(*) as order_count").from_("orders").group_by("user_id")
3232

3333
query = (
34-
sql.select("users.name", "user_orders.order_count")
34+
sql
35+
.select("users.name", "user_orders.order_count")
3536
.with_("user_orders", cte)
3637
.from_("users")
3738
.join("user_orders", "users.id = user_orders.user_id")

docs/examples/usage/usage_query_builder_32.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@ def test_example_32(tmp_path: Path) -> None:
4848

4949
# After: Query Builder
5050
query = (
51-
sql.select("users.id", "users.name", "COUNT(orders.id) as order_count")
51+
sql
52+
.select("users.id", "users.name", "COUNT(orders.id) as order_count")
5253
.from_("users")
5354
.left_join("orders", "users.id = orders.user_id")
5455
.where("users.status = 'active'")

docs/examples/usage/usage_query_builder_5.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ def test_example_5(tmp_path: Path) -> None:
3333

3434
# LEFT JOIN
3535
query = (
36-
sql.select("u.id", "u.name", "COUNT(o.id) as order_count")
36+
sql
37+
.select("u.id", "u.name", "COUNT(o.id) as order_count")
3738
.from_("users u")
3839
.left_join("orders o", "u.id = o.user_id")
3940
.group_by("u.id", "u.name")
@@ -42,7 +43,8 @@ def test_example_5(tmp_path: Path) -> None:
4243

4344
# Multiple JOINs
4445
query = (
45-
sql.select("u.name", "o.id", "p.name as product")
46+
sql
47+
.select("u.name", "o.id", "p.name as product")
4648
.from_("users u")
4749
.join("orders o", "u.id = o.user_id")
4850
.join("order_items oi", "o.id = oi.order_id")

docs/examples/usage/usage_query_builder_7.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ def test_example_7(tmp_path: Path) -> None:
4747

4848
# Multiple aggregations
4949
query = (
50-
sql.select(sql.column("created_at").as_("date"), "COUNT(*) as orders", "SUM(total) as revenue")
50+
sql
51+
.select(sql.column("created_at").as_("date"), "COUNT(*) as orders", "SUM(total) as revenue")
5152
.from_("orders")
5253
.group_by("date")
5354
)

sqlspec/adapters/oracledb/migrations.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ def _get_create_table_sql(self) -> CreateTable:
5151
SQL builder object for Oracle table creation.
5252
"""
5353
return (
54-
sql.create_table(self.version_table)
54+
sql
55+
.create_table(self.version_table)
5556
.column("version_num", "VARCHAR2(32)", primary_key=True)
5657
.column("version_type", "VARCHAR2(16)")
5758
.column("execution_sequence", "INTEGER")
@@ -73,7 +74,8 @@ def _get_current_version_sql(self) -> Select:
7374
SQL builder object for version query.
7475
"""
7576
return (
76-
sql.select('VERSION_NUM AS "version_num"')
77+
sql
78+
.select('VERSION_NUM AS "version_num"')
7779
.from_(self.version_table)
7880
.order_by("EXECUTION_SEQUENCE DESC")
7981
.limit(1)
@@ -90,7 +92,8 @@ def _get_applied_migrations_sql(self) -> Select:
9092
SQL builder object for migrations query.
9193
"""
9294
return (
93-
sql.select(
95+
sql
96+
.select(
9497
'VERSION_NUM AS "version_num"',
9598
'VERSION_TYPE AS "version_type"',
9699
'EXECUTION_SEQUENCE AS "execution_sequence"',

sqlspec/adapters/psqlpy/config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -275,4 +275,4 @@ def get_signature_namespace(self) -> "dict[str, Any]":
275275
def get_event_runtime_hints(self) -> "EventRuntimeHints":
276276
"""Return LISTEN/NOTIFY defaults for Psqlpy adapters."""
277277

278-
return EventRuntimeHints(poll_interval=0.5, select_for_update=True, skip_locked=True, json_passthrough=True)
278+
return EventRuntimeHints(poll_interval=0.5, select_for_update=True, skip_locked=True)

0 commit comments

Comments
 (0)