Skip to content

Commit faf6e87

Browse files
fix: Resolve line length violation in documentation comment
- Break long documentation comment into two lines to meet 88-character limit - All pre-commit checks now pass locally - Unit tests continue to pass (40/40) Co-Authored-By: [email protected] <[email protected]>
1 parent 1178d40 commit faf6e87

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/firebolt_db/firebolt_dialect.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -150,14 +150,13 @@ def create_connect_args(self, url: URL) -> Tuple[List, Dict]:
150150
"""
151151
Build firebolt-sdk compatible connection arguments.
152152
URL format : firebolt://id:secret@host:port/db_name
153-
For Core: firebolt://db_name?url=http://localhost:8080 (full URL in url parameter)
153+
For Core: firebolt://db_name?url=http://localhost:8080
154+
(full URL including scheme, host, port in url parameter)
154155
"""
155156
parameters = dict(url.query)
156157

157158
is_core_connection = "url" in parameters
158-
core_url = (
159-
parameters.pop("url", None) if is_core_connection else None
160-
)
159+
core_url = parameters.pop("url", None) if is_core_connection else None
161160

162161
# parameters are all passed as a string, we need to convert
163162
# bool flag to boolean for SDK compatibility

0 commit comments

Comments
 (0)