Skip to content

Commit 3d1dc9a

Browse files
refactor: remove unnecessary _completed_process attribute storage
- Remove _completed_process field from EntrypointOutput class - No functionality lost as the attribute was never used after assignment - Simplifies the class by not storing unnecessary references - Addresses GitHub PR feedback from @aaronsteers Co-Authored-By: AJ Steers <[email protected]>
1 parent b1e12b6 commit 3d1dc9a

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

airbyte_cdk/test/entrypoint_wrapper.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,6 @@ def __init__(
7777

7878
self._messages: list[AirbyteMessage] | None = None
7979
self._message_file: Path | None = message_file
80-
self._completed_process: Optional[subprocess.CompletedProcess[str]] = None
8180
if messages:
8281
try:
8382
self._messages = [self._parse_message(message) for message in messages]
@@ -309,7 +308,6 @@ def from_completed_process(
309308
else:
310309
instance = cls()
311310

312-
instance._completed_process = completed_process
313311
return instance
314312

315313

0 commit comments

Comments
 (0)