Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

Commit 18fe16d

Browse files
committed
Fix CodeRabbit warnings for previous_leased
1 parent fc1f54d commit 18fe16d

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

packages/jumpstarter/jumpstarter/exporter/exporter.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,13 @@ class Exporter(AsyncContextManagerMixin, Metadata):
132132
AFTER_LEASE_HOOK, BEFORE_LEASE_HOOK_FAILED, AFTER_LEASE_HOOK_FAILED.
133133
"""
134134

135+
_previous_leased: bool = field(init=False, default=False)
136+
"""Previous lease state used to detect lease state transitions.
137+
138+
Tracks whether the exporter was leased in the previous status check to
139+
determine when to trigger before-lease and after-lease hooks.
140+
"""
141+
135142
_lease_context: LeaseContext | None = field(init=False, default=None)
136143
"""Encapsulates all resources associated with the current lease.
137144
@@ -478,7 +485,7 @@ async def serve(self): # noqa: C901
478485
break
479486

480487
# Check for lease state transitions
481-
previous_leased = hasattr(self, "_previous_leased") and self._previous_leased
488+
previous_leased = self._previous_leased
482489
current_leased = status.leased
483490

484491
# Check if this is a new lease assignment (first time or lease name changed)

0 commit comments

Comments
 (0)