Autoupdate pre-commit hooks (#442)

neuro-inc · Dec 19, 2024 · ec022b1 · ec022b1
1 parent 66b9049
commit ec022b1
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 29 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,11 +1,11 @@
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: 'v4.3.0'
+  rev: 'v5.0.0'
   hooks:
   - id: check-merge-conflict
     exclude: "rst$"
 - repo: https://github.com/asottile/yesqa
-  rev: v1.3.0
+  rev: v1.5.0
   hooks:
   - id: yesqa
 - repo: https://github.com/sondrelg/pep585-upgrade
@@ -15,22 +15,21 @@ repos:
     args:
     - --futures=true
 - repo: https://github.com/Zac-HD/shed
-  rev: 0.10.1
+  rev: 2024.10.1
   hooks:
   - id: shed
     args:
     - --refactor
-    - --py39-plus
     types_or:
     - python
     - markdown
     - rst
 - repo: https://github.com/PyCQA/flake8
-  rev: 4.0.1
+  rev: 7.1.1
   hooks:
   - id: flake8
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: 'v4.3.0'
+  rev: 'v5.0.0'
   hooks:
   - id: check-case-conflict
   - id: check-json
@@ -44,14 +43,14 @@ repos:
   - id: debug-statements
 # Another entry is required to apply file-contents-sorter to another file
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: 'v4.3.0'
+  rev: 'v5.0.0'
   hooks:
   - id: file-contents-sorter
     files: |
       docs/spelling_wordlist.txt|
       .gitignore
 - repo: https://github.com/rhysd/actionlint
-  rev: v1.6.15
+  rev: v1.7.4
   hooks:
   - id: actionlint-docker
     args:
@@ -62,7 +61,7 @@ repos:
     - -ignore
     - 'SC1004:'
 - repo: https://github.com/sirosen/check-jsonschema
-  rev: 0.17.1
+  rev: 0.30.0
   hooks:
   - id: check-github-actions
 ci:

diff --git a/neuro_logging/trace.py b/neuro_logging/trace.py
@@ -54,22 +54,21 @@ async def sentry_trace_cm(
     tags: Optional[Mapping[str, str]] = None,
     data: Optional[Mapping[str, Any]] = None,
 ) -> AsyncIterator[Optional[sentry_sdk.tracing.Span]]:
-    with Hub(Hub.current) as hub:
-        with hub.start_span(op="call", description=name) as child:
-            if tags:
-                for key, value in tags.items():
-                    child.set_tag(key, value)
-            if data:
-                for key, value in data.items():
-                    child.set_data(key, value)
-            try:
-                yield child
-            except asyncio.CancelledError:
-                child.set_status("cancelled")
-                raise
-            except Exception as exc:
-                hub.capture_exception(error=exc)
-                raise
+    with Hub(Hub.current) as hub, hub.start_span(op="call", description=name) as child:
+        if tags:
+            for key, value in tags.items():
+                child.set_tag(key, value)
+        if data:
+            for key, value in data.items():
+                child.set_data(key, value)
+        try:
+            yield child
+        except asyncio.CancelledError:
+            child.set_status("cancelled")
+            raise
+        except Exception as exc:
+            hub.capture_exception(error=exc)
+            raise
 
 
 @asynccontextmanager

diff --git a/tests/test_trace.py b/tests/test_trace.py
@@ -21,11 +21,13 @@
 def sentry_transaction() -> t.Iterator[None]:
     sentry_sdk.init(traces_sample_rate=1.0)
     hub = sentry_sdk.Hub.current
-    with sentry_sdk.Hub(hub) as hub:
-        with hub.start_transaction(
+    with (
+        sentry_sdk.Hub(hub) as hub,
+        hub.start_transaction(
             Transaction(name="test", hub=hub, parent_sampled=True, sampled=True)
-        ):
-            yield
+        ),
+    ):
+        yield
 
 
 @pytest.mark.usefixtures("sentry_transaction")